[PATCH] D19950: Use frequency info to guide Loop Invariant Code Motion.

David Li via llvm-commits llvm-commits at lists.llvm.org
Thu May 5 08:42:05 PDT 2016


davidxl added a comment.

A test case should be added to demonstrate the issue.


================
Comment at: test/Other/pass-pipelines.ll:40
@@ -39,1 +39,3 @@
 ; CHECK-O2: Loop Pass Manager
+; CHECK-O2: Branch Probability Analysis
+; CHECK-O2: Block Frequency Analysis
----------------
mcrosier wrote:
> mcrosier wrote:
> > I may be missing something here, but is this test even relevant to this patch?
> I guess this just shows the BPA/BFA is being added to the pipeline?  We also need a test in test/Transforms/LICM..
BPI is required by BFI


http://reviews.llvm.org/D19950





More information about the llvm-commits mailing list