[PATCH] D17078: [LV] Add support for insertelt/extractelt processing during type truncation

James Molloy via llvm-commits llvm-commits at lists.llvm.org
Mon Feb 15 07:08:41 PST 2016


jmolloy accepted this revision.
jmolloy added a comment.
This revision is now accepted and ready to land.

Hi Silviu,

Thanks for fixing this, it's been on my radar for a while. Apart from my comment about using auto*, LGTM.

James


================
Comment at: lib/Transforms/Vectorize/LoopVectorize.cpp:3248
@@ -3244,1 +3247,3 @@
         continue;
+      } else if (InsertElementInst *IE = dyn_cast<InsertElementInst>(I)) {
+        auto Elements = IE->getOperand(0)->getType()->getVectorNumElements();
----------------
Please use auto* for readability.


http://reviews.llvm.org/D17078





More information about the llvm-commits mailing list