[llvm] [AArch64][CostModel] Consider the cost of const vector (PR #117539)

Sushant Gokhale via llvm-commits llvm-commits at lists.llvm.org
Tue Nov 26 06:04:04 PST 2024


sushgokh wrote:

> Thanks for the patch, I planned adding constant vector materialization cost myself for some time. Unfortunately, SLP part of this patch is wrong. The only place, where the constant materialization cost must be used in SLPVectorizer are `InstructionCost getBuildVectorCost(ArrayRef<Value *> VL, Value *Root)` function and `InstructionCost BoUpSLP::getGatherCost` function, all other cost estimations must be in TTI

Thanks @alexey-bataev . Will update the patch.

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


More information about the llvm-commits mailing list