[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