[PATCH] D15512: Polish atomic pointers

JF Bastien via llvm-commits llvm-commits at lists.llvm.org
Mon Dec 14 16:21:38 PST 2015


jfb created this revision.
jfb added a reviewer: reames.
jfb added a subscriber: llvm-commits.

I didn't realize that we already allowed atomic load/store of pointers,
it was added in 2012 by r162146. This patch updates the documentation
and tightens the verifier by using DataLayout to make sure that the
stored size is byte-sized and power-of-two. DataLayout is also used for
integers, and while I'm here I updated the corresponding code for
cmpxchg and rmw.

See the following discussion for context and upcoming changes to
add floating-point and vector atomics:
  https://groups.google.com/forum/#!topic/llvm-dev/Nh0P_E3CRoo/discussion

http://reviews.llvm.org/D15512

Files:
  docs/LangRef.rst
  lib/IR/Verifier.cpp

-------------- next part --------------
A non-text attachment was scrubbed...
Name: D15512.42798.patch
Type: text/x-patch
Size: 8000 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20151215/820ad9de/attachment.bin>


More information about the llvm-commits mailing list