[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 16:48:59 PDT 2018


sguggill marked 3 inline comments as done.
sguggill added inline comments.


================
Comment at: lib/Transforms/Vectorize/LoopVectorize.cpp:2800
 void InnerLoopVectorizer::emitMemRuntimeChecks(Loop *L, BasicBlock *Bypass) {
+  // VPlan-native path does not do any analysis for runtime checks currently.
+  if (EnableVPlanNativePath)
----------------
fhahn wrote:
> Should we ever hit this code path in the vplan native path at the moment? If not, I would turn it into an assert.
We do hit this code in vplan native path and later hit a seg fault as getLAI returns NULL. 


Repository:
  rL LLVM

https://reviews.llvm.org/D50820





More information about the llvm-commits mailing list