[PATCH] D80061: [WebAssembly] Handle exception specifications

[Heejin Ahn via Phabricator via cfe-commits]

aheejin created this revision.
aheejin added a reviewer: dschuff.
Herald added subscribers: cfe-commits, sunfish, jgravelle-google, sbc100.
Herald added a project: clang.
aheejin edited the summary of this revision.
aheejin updated this revision to Diff 264423.
aheejin added a comment.

- rst file fix


aheejin added a comment.

This is an improved version of D79655 <https://reviews.llvm.org/D79655>. The warning now can be suppressed by `-Wno-wasm-exception-spec`. To do this, we needed a diagnostic group.


Wasm currently does not fully handle exception specifications. Rather
than crashing,

- This treats `throw()` in the same way as `noexcept`.
- This ignores and prints a warning for `throw(type, ..)`, for a temporary measure. This warning is controlled by `-Wwasm-exception-spec`, which is on by default. You can suppress the warning by using `-Wno-wasm-exception-spec`.


Repository:
  rG LLVM Github Monorepo

https://reviews.llvm.org/D80061

Files:
  clang/docs/DiagnosticsReference.rst
  clang/include/clang/Basic/DiagnosticGroups.td
  clang/include/clang/Basic/DiagnosticSemaKinds.td
  clang/lib/CodeGen/CGException.cpp
  clang/test/CodeGenCXX/wasm-eh.cpp

-------------- next part --------------
A non-text attachment was scrubbed...
Name: D80061.264423.patch
Type: text/x-patch
Size: 6859 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/cfe-commits/attachments/20200516/486a2a65/attachment-0001.bin>