[PATCH] D86510: [compiler-rt] Fix atomic support functions on 32-bit architectures

Alexander Richardson via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Fri Sep 4 09:38:33 PDT 2020


arichardson updated this revision to Diff 289985.
arichardson added a comment.

Use the lock-free code when

  a) atomic ops of that size are always lock-free
  b) aligned atomics ops are lock-free and the pointer is aligned

Note: the optimized functions assume that the pointer is aligned since the argument is a uint64_t, etc. If this is not correct the signature should be changed to use void* instead.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D86510

Files:
  compiler-rt/lib/builtins/atomic.c

-------------- next part --------------
A non-text attachment was scrubbed...
Name: D86510.289985.patch
Type: text/x-patch
Size: 9518 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20200904/7ebda44e/attachment.bin>


More information about the llvm-commits mailing list