[PATCH] D142895: [VPlan] Move mayHaveSideeffects for FORs check to VPlan.

Mon Apr 17 02:25:16 PDT 2023

peterwaller-arm added a comment.

Doesn't build for me, tried against main (8bf7f86d7966 <https://reviews.llvm.org/rG8bf7f86d7966ff835967669fb6bf23897e20b71d>) and your latest commit to vplan (668045eb7762 <https://reviews.llvm.org/rG668045eb77628be13e448ffbb855473ffca1cc43>).

  ../../llvm/lib/Transforms/Vectorize/VPlanTransforms.cpp:791:27: error: no member named 'getOrAddVPValue' in 'llvm::VPlan'; did you mean 'getVPValue'?
      VPValue *Start = Plan.getOrAddVPValue(ID.getStartValue());
                            ^~~~~~~~~~~~~~~
                            getVPValue
  ../../llvm/lib/Transforms/Vectorize/VPlanTransforms.cpp:794:21: error: no matching constructor for initialization of 'llvm::VPWidenIntOrFpInductionRecipe'
      auto *Ind = new VPWidenIntOrFpInductionRecipe(Phi, Start, Step, ID);
                      ^                             ~~~~~~~~~~~~~~~~~~~~

Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D142895