[PATCH] D149486: [RISCV] Strengthen atomic ordering for sequentially consistent stores

[Alex Bradbury via Phabricator via llvm-commits]

asb added a comment.

In D149486#4404354 <https://reviews.llvm.org/D149486#4404354>, @paulkirth wrote:

> @asb, @jrtc27, @craig.topper,  since psABI has merged https://github.com/riscv-non-isa/riscv-elf-psabi-doc/pull/378, are there any issues that we still need to iron out with the current approach?
>
> I assume we still want this off by default until at least the ELF attributes thing is ironed out, but the //experimental// moniker can probably just get dropped since its in the ABI now.

Sounds like a sensible path forward to me. Drop the "experimental" moniker and I think we can land this option, IMHO.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D149486