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



_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Reply via email to