[PATCH] D47553: Add TargetLowering::shouldExpandAtomicToLibCall and query it from AtomicExpandPass

[Alex Bradbury via Phabricator via llvm-commits]

asb added a subscriber: theraven.
asb added a comment.

I'm going to un-abandon this patch. This hook is still helpful in order to e.g. lower to __atomic libcalls for all 8 and 16-bit operations but not 32-bit. This seems acceptable based on the linked GCC and LLVM docs, but isn't possible with the current MaxSizeInBitsSupported. Am I missing any problems with that strategy?

It seems GCC RISC-V will emit `__atomic_*` libcalls for 8 and 16-bit operations other than load/store even with -march=rv32ia. The immediate response in the linked GCC bug report is that perhaps that fine if the RISC-V version of these sub-word atomics is guaranteed to be lock-free (which apparently is the case for libatomic compiled for rv32ia). If this is confirmed, it would give a case where intermixing is safe. I don't know how reasonable it is to declare `__atomic_*` libcalls are lock-free if compiled with support for the A extension - if you have views please do jump on the GCC bug report thread <https://gcc.gnu.org/bugzilla/show_bug.cgi?id=86005>.


Repository:
  rL LLVM

https://reviews.llvm.org/D47553