[PATCH] D97136: [VPlan] Support to widen select intructions in VPlan native path

Mauri Mustonen via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Tue Feb 23 23:53:40 PST 2021


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


================
Comment at: llvm/lib/Transforms/Vectorize/VPlanTransforms.cpp:20
 void VPlanTransforms::VPInstructionsToVPRecipes(
-    Loop *OrigLoop, VPlanPtr &Plan,
+    PredicatedScalarEvolution &PSE, Loop *OrigLoop, VPlanPtr &Plan,
     LoopVectorizationLegality::InductionList &Inductions,
----------------
fhahn wrote:
> We only need `SE`, can you pass that directly?
Thanks for the comment! I didn't pay attention enough for this. Removing PSE also cleaned up code in tests as well.


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

https://reviews.llvm.org/D97136



More information about the llvm-commits mailing list