[PATCH] Atomics: emit cmpxchg instructions with weaker failure-order

JF Bastien jfb at chromium.org
Thu Mar 13 11:13:36 PDT 2014


  Ooh yeah, that test case is quite hairy! Sorry about that.

  lgtm :)


================
Comment at: lib/CodeGen/CGAtomic.cpp:787
@@ -703,2 +786,3 @@
+                 Size, Align, llvm::Acquire);
     Builder.CreateBr(ContBB);
     SI->addCase(Builder.getInt32(1), AcquireBB);
----------------
It could be a different change, but the same AtomicOrderingKind change would be more readable below.


http://llvm-reviews.chandlerc.com/D3053



More information about the cfe-commits mailing list