[PATCH] D71025: [LoopFusion] Restrict loop fusion to rotated loops.

Whitney Tsang via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Mon Dec 16 09:39:35 PST 2019


Whitney added inline comments.


================
Comment at: llvm/test/Transforms/LoopFusion/cannot_fuse.ll:42
+bb34:
+  %cmp = icmp slt i32 %N, 50
+  br i1 %cmp, label %bb16, label %bb33
----------------
kbarton wrote:
> Whitney wrote:
> > This guard was not in the original test case, now the second loop has two guards, is that intensional?                    
> Yes, this is intentional to make the two loops not control-flow-equivalent, which is what this specific test is checking. 
The two loops should already be not control-flow-equivalent, as bb7 is not guarded, while bb22 is guarded. Why is the extra guard needed?


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D71025/new/

https://reviews.llvm.org/D71025





More information about the llvm-commits mailing list