[PATCH] D51837: Move a transformation routine from LoopUtils to LoopVectorize.
Vikram TV via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Sun Sep 9 23:07:15 PDT 2018
tvvikram marked an inline comment as done.
tvvikram added inline comments.
================
Comment at: Transforms/Vectorize/LoopVectorize.cpp:2850
+ assert(Step->getType()->isFloatingPointTy() && "Expected FP Step value");
+ auto InductionBinOp = ID.getInductionBinOp();
+ assert(InductionBinOp &&
----------------
dmgreen wrote:
> Nit: I tend to avoid auto in lines that don't already make the type obvious. This is just me though, so feel free to ignore.
I am preferring "auto" :)
Repository:
rL LLVM
https://reviews.llvm.org/D51837
More information about the llvm-commits
mailing list