[PATCH] D67208: [WebAssembly] Add -fwasm-exceptions for wasm EH

[Derek Schuff via Phabricator via cfe-commits]

dschuff added a comment.

If this is to be like `-fdwarf-exceptions` I assume the idea is that eventually it will default on when targeting wasm, and become one of those flags that most users never set. In that case it really just selects the default exception model, and thus it should be compatible with `-fno-exceptions` just as `-fdwarf-exceptions`.
Also `-fno-exceptions` doesn't really mean "no exceptions whatsoever" because if you call something that the compiler isn't sure never throws, it generates an implicit catch-all that calls `std::terminate`. So how it does that would still be affected by the exception model. (and whatever downstream invoke-removing pass or postprocessing tool might care).


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D67208/new/

https://reviews.llvm.org/D67208