[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