[PATCH] D155459: [AArch64] Change the cost of vector insert/extract to 2

Sjoerd Meijer via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Mon Jul 17 07:21:38 PDT 2023


SjoerdMeijer added inline comments.


================
Comment at: llvm/test/Analysis/CostModel/AArch64/bswap.ll:45
 }
 
 define <2 x i32> @bswap_v2i32(<2 x i32> %a) {
----------------
Just a bit of a drive by question first. It's not really caused by this change, I think, but it looks like the cost modelling was already a bit off for these bswaps? 

https://godbolt.org/z/d1s4ToP1G

Or am I missing something?


CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D155459/new/

https://reviews.llvm.org/D155459



More information about the llvm-commits mailing list