[PATCH] D132185: [TTI][AArch64] Update vector extract cost for Neoverse-N1.

David Li via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Fri Aug 19 12:35:55 PDT 2022


davidxl added a comment.

@dmgreen  The extract cost's default value and how the cost is used are two orthogonal things. I suspect that the regression you saw with cost == 2 might be the cause of a hidden bug somewhere else -- in other words, setting it to 3 simply papers over the real issue.

I think this patch is probably the right way to go.  An alternative is to introduce an internal option to control the default value.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D132185



More information about the llvm-commits mailing list