[PATCH] D141602: [TTI][AArch64] Cost model insertelement and indexed LD1 instructions
Dave Green via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Mon Jan 23 01:06:26 PST 2023
dmgreen added a comment.
Im happy for the change to go in, I think. It changes the total cost of load+insert from 1+3 to 1+4 which isnt a huge increase. It sounds OK to me, although it was already fairly high before and there is a chance we might want to adjust it in the future if we have good reason.
Can you change it to `ST->getVectorInsertExtractBaseCost() + 1`?
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D141602/new/
https://reviews.llvm.org/D141602
More information about the llvm-commits
mailing list