[PATCH] D26083: [LV] Scalarize operands of predicated instructions

Gil Rapaport via llvm-commits llvm-commits at lists.llvm.org
Mon Oct 31 14:40:41 PDT 2016


gilr added inline comments.


================
Comment at: lib/Transforms/Vectorize/LoopVectorize.cpp:6546
+    // there's nothing to do.
+    if (ScalarCosts.count(I) || !TheLoop->contains(I))
+      continue;
----------------
It seems the only instructions pushed into Worklist are PredInst and instructions from its basic block. Is the invariance check necessary?


https://reviews.llvm.org/D26083





More information about the llvm-commits mailing list