[PATCH] D155517: [RISC-V] Add proposed mapping for Ztso

Alex Bradbury via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Thu Jul 20 06:56:30 PDT 2023


asb added a comment.

@patrick-rivos: I've just posted D155840 <https://reviews.llvm.org/D155840> to introduce a field for RISCVSubtarget in those pseudo expansion passes. I think that would probably be a good approach here, but you might want to wait for any feedback on that review before changing again in case others disagree with me!

LGTM once we resolve that (I'm expecting a quick review on that linked patch - if it's slow for some reason let's loop back and make sure this patch isn't blocked on it unreasonably). Thanks for such a rapid update, and of course for updating us to the most recent proposed mappings.


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

https://reviews.llvm.org/D155517



More information about the llvm-commits mailing list