[PATCH] D107530: [WebAssembly] Don't do SjLj transformation when there's only setjmp

[Derek Schuff via Phabricator via llvm-commits]

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:744
 
+  if (EnableEmSjLj) {
+    if (SetjmpF) {
----------------
can this be collapsed into `if (EnableEmSjLj && SetjmpF)`?


================
Comment at: llvm/lib/Target/WebAssembly/WebAssemblyLowerEmscriptenEHSjLj.cpp:749
+        Function *UserF = cast<Instruction>(U)->getFunction();
+        // If we a function that calls setjmp does not contain any other calls
+        // that can longjmp, we don't need to do any transformation on that
----------------



Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D107530