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
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
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
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
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
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