[PATCH] D114951: [Analysis][AArch64] Add on the address computational cost for gathers/scatters

[Peter Waller via Phabricator via llvm-commits]

peterwaller-arm added inline comments.


================
Comment at: llvm/lib/Target/AArch64/AArch64TargetTransformInfo.cpp:1791
+  MemOpCost += 1;
   return LT.first * MemOpCost * getMaxNumElements(LegalVF);
 }
----------------
This appears to be increasing the cost by a factor of the legalization cost, in addition to the 1 x element count as stated in the PR body, is that intended? I might instead have expected this to read `return (LT.first * MemOpCost + 1) * getMaxNumElements(LegalVF)`.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D114951