[PATCH] D130621: [RISCV] Add target feature to force-enable atomics

[Alex Bradbury via Phabricator via llvm-commits]

asb added a comment.

Broadly speaking, this looks good to me (thanks!). I think in some previous discussions on this topic I hadn't appreciated the lock-free nature of the __sync calls vs __atomic.

My one request would be to flesh out the test coverage so the files cover all of the different atomicrmw operations - given they all had to be explicitly listed in the `setOperationAction` call to avoid their expansion, it would be worth verifying this (to provide some resilience against future refactorings).


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

https://reviews.llvm.org/D130621