[PATCH] D69075: [WebAssembly] -pthread implies -target-feature +sign-ext

2019-10-17 Thread Thomas Lively via Phabricator via cfe-commits
This revision was automatically updated to reflect the committed changes. tlively marked an inline comment as done. Closed by commit rG807cecad5d98: [WebAssembly] -pthread implies -target-feature +sign-ext (authored by tlively). Changed prior to commit:

[PATCH] D69075: [WebAssembly] -pthread implies -target-feature +sign-ext

2019-10-16 Thread Derek Schuff via Phabricator via cfe-commits
dschuff accepted this revision. dschuff added inline comments. This revision is now accepted and ready to land. Comment at: clang/test/Driver/wasm-toolchain.c:76 +// '-pthread' not allowed with '-mno-mutable-globals' +// RUN: %clang -### -no-canonical-prefixes -target

[PATCH] D69075: [WebAssembly] -pthread implies -target-feature +sign-ext

2019-10-16 Thread Thomas Lively via Phabricator via cfe-commits
tlively created this revision. tlively added reviewers: aheejin, dschuff. Herald added subscribers: cfe-commits, jfb, sunfish, jgravelle-google, sbc100. Herald added a project: clang. The sign extension proposal was motivated by a desire to not have separate sign-extending atomic operations, so