[PATCH] D28381: [WebAssembly] Always inline atomics
Dan Gohman via Phabricator via cfe-commits
cfe-commits at lists.llvm.org
Fri Jan 6 09:19:06 PST 2017
sunfish added a comment.
(Emscripten does set the thread model.)
Whether WebAssembly should provide 64-bit atomics when on hardware where they're not lock-free is an interesting wasm design question, and the code here and in target-independent parts of clang may need to be changed as a result (right now, clang's target-independent code automatically defines __GCC_ATOMIC_LLONG_LOCK_FREE to 2 with this patch, which is incorrect).
So that's a separate topic. I'd like to keep changes meant to fix the -mthread-model single case.
https://reviews.llvm.org/D28381
More information about the cfe-commits
mailing list