[PATCH] D57874: [WebAssembly] Make thread-related options consistent
Heejin Ahn via Phabricator via cfe-commits
cfe-commits at lists.llvm.org
Fri Feb 8 17:53:22 PST 2019
aheejin updated this revision to Diff 186088.
aheejin marked 8 inline comments as done.
aheejin added a comment.
I had an offline discussion with @tlively and @dschuff and decided to remove `-atomics` option in the driver. Instead, `clang -cc1`'s `-target-feature +atomics` will be either of `-pthread` or `-mthread-model posix` is given. This is to reduce the total number of options and thus the total number of combinations of options. Sorry for frequent changes. Also addressed comments.
CHANGES SINCE LAST ACTION
-------------- next part --------------
A non-text attachment was scrubbed...
Size: 7186 bytes
Desc: not available
More information about the cfe-commits