[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