[PATCH] D18200: Add __atomic_* lowering to AtomicExpandPass.

JF Bastien via llvm-commits llvm-commits at lists.llvm.org
Mon Apr 4 13:37:56 PDT 2016


jfb added inline comments.

================
Comment at: include/llvm/CodeGen/RuntimeLibcalls.h:401
@@ -400,1 +400,3 @@
 
+    // New style atomics.
+    ATOMIC_LOAD,
----------------
"new" is a comment that won't age well. Could you instead comment on SYNC_ saying ATOMIC_ are the newer approach, and refer to docs?


http://reviews.llvm.org/D18200





More information about the llvm-commits mailing list