[PATCH] D16382: Add LoopSimplifyCFG pass

Michael Zolotukhin via llvm-commits llvm-commits at lists.llvm.org
Wed Jan 20 17:37:29 PST 2016


mzolotukhin added a comment.

Hi,

First of all, thanks for working on this! One question on the test below.

Michael


================
Comment at: test/Transforms/LoopSimplifyCFG/merge-header.ll:4-5
@@ +3,4 @@
+; CHECK-LABEL: foo
+; CHECK: entry
+; CHECK-NEXT: br label %inner
+define i32 @foo(i32* %P, i64* %Q) {
----------------
Which basic blocks do we merge here? From the check-lines it looks like we merge `%entry` with `%outer`, which doesn't sound right, since we want to merge only blocks inside loops. Am I misreading something here?


Repository:
  rL LLVM

http://reviews.llvm.org/D16382





More information about the llvm-commits mailing list