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

Alex Bradbury via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Mon Aug 1 03:43:43 PDT 2022


asb added a comment.

It might also be worth patching the __sync section in the atomics docs <https://llvm.org/docs/Atomics.html#libcalls-sync> to mention that some targets support this forced-atomics feature and provide a bit more insight on the use case (" for various reasons, it is not practical to emit the instructions inline." is a little vague).

That can be a separate patch, but I've found the LLVM atomics guide to be a really helpful reference so it would be great to keep it as up to date and informative as possible.


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

https://reviews.llvm.org/D130621



More information about the llvm-commits mailing list