[PATCH] D143076: [RISCV] Implement a proposed mapping for Ztso

[Alex Bradbury via Phabricator via llvm-commits]

asb accepted this revision.
asb added a comment.
This revision is now accepted and ready to land.

The code changes LGTM.

As this has been floated in the RISC-V LLVM contributor call, and it's behind an experimental flag anyway (i.e. we can still change tack if future discussion shows there's huge opposition to the proposed lowering), I've got no opposition to merging this sooner rather than later and don't know of other ztso stakeholders working upstream who might be inconvenienced in either case.

Is your preference to try to get this merged in the near future, or would you rather wait for more feedback on the lowerings? (I'm pretty agnostic, for the reasons above).


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

https://reviews.llvm.org/D143076