[PATCH] D140408: [RISCV] Implement isMultiStoresCheaperThanBitsMerge hook
Philip Reames via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Mon Jan 30 10:56:04 PST 2023
reames accepted this revision.
reames added a comment.
This revision is now accepted and ready to land.
LGTM
This looks entirely reasonable. I don't think this is worth gating on ABI. If we did gate it, we'd probably do so on the conversion cost - which would in turn indirectly dependent on ABI. We can revisit that in the future if we ever care about the non-F non-D configuration performance in this specific case.
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D140408/new/
https://reviews.llvm.org/D140408
More information about the llvm-commits
mailing list