[PATCH] D109375: [WebAssembly] Error out on indirect uses of setjmp

Derek Schuff via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Tue Sep 7 14:05:02 PDT 2021


dschuff accepted this revision.
dschuff added inline comments.
This revision is now accepted and ready to land.


================
Comment at: llvm/lib/Target/WebAssembly/WebAssemblyLowerEmscriptenEHSjLj.cpp:881
+        SS << *U;
+        report_fatal_error(Twine("Does not support indirect uses of setjmp: ") +
+                           SS.str());
----------------
This error could probably be a bit clearer. e.g. what exactly does not support indirect use of setjmp? Maybe "wasm does not support indirect use of setjmp" or even just "indirect use of setjmp is not supported"


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D109375



More information about the llvm-commits mailing list