[PATCH] D116288: [VPlan] Add recipe to handle SCEV expansion (NFC).

[Ayal Zaks via Phabricator via llvm-commits]

Ayal accepted this revision.
Ayal added inline comments.
This revision is now accepted and ready to land.


================
Comment at: llvm/lib/Transforms/Vectorize/VPlan.cpp:1359
 
+void VPExpandSCEVRecipe::execute(VPTransformState &State) {
+  const DataLayout &DL =
----------------
Set the single value per part? (Tests with UF>1 work?)
Assert it's not called per lane.


================
Comment at: llvm/lib/Transforms/Vectorize/VPlan.h:1632
+/// TODO: Currently the recipe always expands the expression in the loop
+/// pre-header.
+class VPExpandSCEVRecipe : public VPRecipeBase, public VPValue {
----------------
but the recipe is currently placed in the header; place it in the pre-header once the latter is modelled in VPlan as a VPBasicBlock.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D116288