[PATCH] D20932: [LV] Use vector vphis for more induction variables

Elena Demikhovsky via llvm-commits llvm-commits at lists.llvm.org
Tue Jun 7 23:52:35 PDT 2016


delena added inline comments.

================
Comment at: lib/Transforms/Vectorize/LoopVectorize.cpp:2122
@@ -2121,3 +2121,3 @@
     Step = ConstantInt::getSigned(TruncType, Step->getSExtValue());
     Start = Builder.CreateCast(Instruction::Trunc, Start, TruncType);
   }
----------------
I thought, that you can put here Builder.CreateSExtOrTrunc(Start, TruncType) , Start and Step should have the  same type anyway.
You can change name from TruncType to CastType.

But if I'm wrong, you can submit this patch as is.


http://reviews.llvm.org/D20932





More information about the llvm-commits mailing list