[PATCH] D22711: Diagnose invalid failure memory orderings.

Eric Fiselier via cfe-commits cfe-commits at lists.llvm.org
Fri Jul 22 17:49:37 PDT 2016


EricWF created this revision.
EricWF added reviewers: majnemer, jfb, rsmith.
EricWF added a subscriber: cfe-commits.

Clang currently issues diagnostics if the success memory order argument is invalid, but it doesn't check the failure ordering argument for cmpxchg atomic operations.
This patch adds diagnostics if the failure ordering is either `memory_order_release` or `memory_order_acq_rel`.

https://reviews.llvm.org/D22711

Files:
  lib/Sema/SemaChecking.cpp
  test/Sema/atomic-ops.c

-------------- next part --------------
A non-text attachment was scrubbed...
Name: D22711.65200.patch
Type: text/x-patch
Size: 6735 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/cfe-commits/attachments/20160723/0061ddde/attachment.bin>


More information about the cfe-commits mailing list