[PATCH] D108582: [WebAssembly] Add Wasm SjLj option support for clang

2021-08-24 Thread Heejin Ahn via Phabricator via cfe-commits
This revision was landed with ongoing or failed builds. This revision was automatically updated to reflect the committed changes. Closed by commit rGa947b40cafa8: [WebAssembly] Add Wasm SjLj option support for clang (authored by aheejin). Repository: rG LLVM Github Monorepo CHANGES SINCE LAST

[PATCH] D108582: [WebAssembly] Add Wasm SjLj option support for clang

2021-08-24 Thread Heejin Ahn via Phabricator via cfe-commits
aheejin updated this revision to Diff 368519. aheejin added a comment. This revision is now accepted and ready to land. Remove multivalue setting Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D108582/new/ https://reviews.llvm.org/D108582 Files: c

[PATCH] D108582: [WebAssembly] Add Wasm SjLj option support for clang

2021-08-24 Thread Heejin Ahn via Phabricator via cfe-commits
aheejin added a comment. As we discussed offline, I'll remove `+multivalue` part for now; it doesn't have reliable support in the Wasm LLVM backend and Binaryen yet. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D108582/new/ https://reviews.llvm.or

[PATCH] D108582: [WebAssembly] Add Wasm SjLj option support for clang

2021-08-23 Thread Heejin Ahn via Phabricator via cfe-commits
aheejin planned changes to this revision. aheejin added a comment. Oh sorry I clicked the wrong button Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D108582/new/ https://reviews.llvm.org/D108582 ___ c

[PATCH] D108582: [WebAssembly] Add Wasm SjLj option support for clang

2021-08-23 Thread Heejin Ahn via Phabricator via cfe-commits
aheejin accepted this revision. aheejin added a comment. This revision is now accepted and ready to land. Oh, I fortot I made this currently set `+multivalue` feature... Maybe we have to rethink that. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D1

[PATCH] D108582: [WebAssembly] Add Wasm SjLj option support for clang

2021-08-23 Thread Heejin Ahn via Phabricator via cfe-commits
aheejin created this revision. aheejin added reviewers: dschuff, tlively. Herald added subscribers: wingo, ecnelises, sunfish, jgravelle-google, sbc100. aheejin requested review of this revision. Herald added a project: clang. Herald added a subscriber: cfe-commits. This adds support for Wasm SjLj