[PATCH] D58742: [WebAssembly] Remove uses of ThreadModel
Dan Gohman via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Wed Feb 27 17:46:44 PST 2019
sunfish added a comment.
This is still a little confusing to me. -matomic is supposed to be a subtarget flag, stating that the wasm implementation we will run on supports atomic instructions. -mthread-model posix is about the C++ interpretation -- what style implementation of memory model do we want? In the future, -matomic may become enabled by default, when enough wasm engines generally support it. However, -mthread-model single/posix may still be useful to control independently, because even with wasm engines supporting atomic, there are reasons users might still want to compile their apps single-threaded: access to linear memory with no declared max, lower overall code size, or other things.
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D58742/new/
https://reviews.llvm.org/D58742
More information about the llvm-commits
mailing list