[PATCH] D11241: Create a wrapper pass for BranchProbabilityInfo so that branch probability analysis can be done without requiring a pass.

Cong Hou congh at google.com
Wed Jul 15 15:39:08 PDT 2015


Thanks for the review!


thanks,
Cong


On Wed, Jul 15, 2015 at 3:27 PM, Duncan P. N. Exon Smith
<dexonsmith at apple.com> wrote:
>
>> On 2015-Jul-15, at 14:14, Cong Hou <congh at google.com> wrote:
>>
>> congh created this revision.
>> congh added a reviewer: dexonsmith.
>> congh added subscribers: llvm-commits, davidxl.
>>
>> This patch is similar to the one adding a wrapper to BlockFrequencyInfo. The patch creates a wrapper pass for BranchProbabilityInfo. This is useful when we want to do branch probability analysis conditionally (e.g. only in PGO mode) but don't want to add one more pass dependence.
>>
>> http://reviews.llvm.org/D11241
>>
>> Files:
>>  include/llvm/Analysis/BranchProbabilityInfo.h
>>  include/llvm/InitializePasses.h
>>  lib/Analysis/Analysis.cpp
>>  lib/Analysis/BlockFrequencyInfo.cpp
>>  lib/Analysis/BranchProbabilityInfo.cpp
>>  lib/CodeGen/SelectionDAG/SelectionDAGISel.cpp
>>  lib/Transforms/Scalar/InductiveRangeCheckElimination.cpp
>>
>> <D11241.29829.patch>
>
> LGTM.
>




More information about the llvm-commits mailing list