[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