[PATCH] D147783: [VPlan] Add stride->constant VPlan mapping at construction.

[Florian Hahn via Phabricator via llvm-commits]

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


================
Comment at: llvm/lib/Transforms/Vectorize/LoopVectorize.cpp:8911
   VPBasicBlock *Preheader = new VPBasicBlock("vector.ph");
   auto Plan = std::make_unique<VPlan>(Preheader);
 
----------------
peixin wrote:
> Don't get why this cannot capture the cases in preheader?
The issue is that the strides in the preheader come from SCEV expressions, as @reames mentioned and that's not yet captured in the VPlan scope. 

There's another patch (D147965) that moves step expansion used in the pre-header to VPlan as well. We should be then able to take care of this here as well.


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D147783/new/

https://reviews.llvm.org/D147783