[llvm] [LV] Add VPWiden{Load|Store}EVLRecipe into emitInvalidCostRemarks(). (PR #119708)
Florian Hahn via llvm-commits
llvm-commits at lists.llvm.org
Tue Dec 17 06:24:07 PST 2024
fhahn wrote:
> > If the assumption is we cannot encounter those, could it be an assert instead? Not sure if the type switch would already assert.
>
> This issue already crashed our downstream compiler by invalid cost of VPWidenLoadEVLRecipe and VPWidenStoreEVLRecipe.
>
> I will merge this PR after adding a test case that will trigger this issue in upstream, but the cost model difference makes it hard to find.
The assert may help finding a test case upstream.
https://github.com/llvm/llvm-project/pull/119708
More information about the llvm-commits
mailing list