[llvm] [LV] Add VPWiden{Load|Store}EVLRecipe into emitInvalidCostRemarks(). (PR #119708)

Elvis Wang via llvm-commits llvm-commits at lists.llvm.org
Tue Dec 17 06:22:21 PST 2024


ElvisWang123 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.


https://github.com/llvm/llvm-project/pull/119708


More information about the llvm-commits mailing list