[PATCH] IR: add failure ordering to the cmpxchg instruction

Chandler Carruth chandlerc at gmail.com
Mon Mar 10 14:01:51 PDT 2014


  This looks good to me, with the provision that we should probably bump the bitcode version.

  While this change is *backwards* compatible to old bitcodes, it is not forwards compatible. Put another way, old bitcode readers will stop working, and so we should pus the version number forward so they get a version error rather than a structural error.

  I'm fine with you committing this briefly until you hear back from someone about the bitcode version -- until we make a release, nothing like that is set in stone, so we can fix the bitcode versions #s.


================
Comment at: include/llvm/CodeGen/SelectionDAGNodes.h:1096
@@ +1095,3 @@
+
+
+  void InitAtomic(AtomicOrdering SuccessOrdering,
----------------
Spurious blank line?


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



More information about the llvm-commits mailing list