[PATCH] D130480: [ARM] Add target feature to force 32-bit atomics

Eli Friedman via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Mon Jul 25 12:05:19 PDT 2022


efriedma added a comment.

> the ability to override setMaxAtomicSizeInBitsSupported() would also be useful in general...

Probably for each target where you want a feature like this, you need to go through and ensure it actually works.  Just overriding setMaxAtomicSizeInBitsSupported() on a target that isn't expecting it is going to either crash, or generate `__sync_*` calls for load/store ops.


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

https://reviews.llvm.org/D130480



More information about the llvm-commits mailing list