[PATCH] D50820: [VPlan] Implement initial vector code generation support for simple outer loops.

Satish K Guggilla via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Thu Aug 30 17:22:02 PDT 2018


sguggill marked an inline comment as done.
sguggill added inline comments.


================
Comment at: lib/Transforms/Vectorize/LoopVectorize.cpp:7400
+  // outer loops - a check for empty loop is sufficient here.
+  if (L->empty())
+    return false;
----------------
fhahn wrote:
> as it is currently, we should only get to this point for outer loops, right? Then this could be an assert.
Yes, that is right. I have removed this code as the call to this function is guarded by a check for non-empty loops.


Repository:
  rL LLVM

https://reviews.llvm.org/D50820





More information about the llvm-commits mailing list