[PATCH] D58742: [WebAssembly] Remove uses of ThreadModel

Dan Gohman via Phabricator via cfe-commits cfe-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.

  rG LLVM Github Monorepo



More information about the cfe-commits mailing list