[llvm] [VPlan][NFC] Add cost for `VPWidenMemoryRecipe`. (PR #105614)
Elvis Wang via llvm-commits
llvm-commits at lists.llvm.org
Sun Sep 1 17:12:35 PDT 2024
ElvisWang123 wrote:
> > > > With the change it should also be possible to remove `VPWidenMemoryRecipe` from `getInstructionForCost` in `VPlan.cpp`
> > >
> > >
> > > Still missing?
> >
> >
> > Remove `VPWidenMemoryRecipe` from `getInstructionForCost` will cause a test fail due to different VF between legacy model. Maybe leave a TODO behind to revisit it when we don't need to compare to the legacy model?
>
> yes that would be great, together with an explanation that VPWidenMemoryRecipe case in getInstructionForCost still being needed for force-target-instruction handling.
Done, thanks!
https://github.com/llvm/llvm-project/pull/105614
More information about the llvm-commits
mailing list