[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