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
_______________________________________________
cfe-commits mailing list
[email protected]
http://lists.cs.uiuc.edu/mailman/listinfo/cfe-commits

Reply via email to