[PATCH] D109669: [WebAssembly] Handle _setjmp and _longjmp in SjLj

Heejin Ahn via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Mon Sep 13 14:00:59 PDT 2021


aheejin added inline comments.


================
Comment at: llvm/lib/Target/WebAssembly/WebAssemblyLowerEmscriptenEHSjLj.cpp:838
+  Function *SetjmpF2 = M.getFunction("_setjmp");
+  Function *LongjmpF2 = M.getFunction("_longjmp");
+  if (SetjmpF2) {
----------------
I wanted to name these variable `_SetjmpF` of `SetjmpF`, but clang-tidy didn't like it...


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D109669



More information about the llvm-commits mailing list