[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