[PATCH] D24009: [LoopVectorizer] Support predicating instructions in blocks with several input edges

Michael Kuperstein via llvm-commits llvm-commits at lists.llvm.org
Tue Aug 30 08:11:10 PDT 2016


mkuper added inline comments.

================
Comment at: test/Transforms/LoopVectorize/if-pred-non-void.ll:197
@@ +196,3 @@
+
+if.then:                                          ; preds = %for.body
+  %sd1 = sdiv i32 %psd, %lsd
----------------
mssimpso wrote:
> Is the test checking what you want it to? I was expecting if.then to have more than one predecessor (and trigger the assert) since it's the block with the sdiv's.
You're completely right - I was tweaking the test to make the checks more readable, and didn't notice I no longer had the right structure.
Will fix it, thanks!


https://reviews.llvm.org/D24009





More information about the llvm-commits mailing list