[PATCH] D28381: [WebAssembly] Always inline atomics

Derek Schuff via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Fri Jan 6 08:35:17 PST 2017


dschuff added inline comments.


================
Comment at: test/Preprocessor/init.c:8613
 // WEBASSEMBLY32-NEXT:#define __GCC_ATOMIC_INT_LOCK_FREE 2
-// WEBASSEMBLY32-NEXT:#define __GCC_ATOMIC_LLONG_LOCK_FREE 1
+// WEBASSEMBLY32-NEXT:#define __GCC_ATOMIC_LLONG_LOCK_FREE 2
 // WEBASSEMBLY32-NEXT:#define __GCC_ATOMIC_LONG_LOCK_FREE 2
----------------
I'm not sure we actually want to set this; it's possible that even if 64-bit atomics are specified by the wasm platform we may not promise that they have to be lock-free.


https://reviews.llvm.org/D28381





More information about the cfe-commits mailing list