[PATCH] D118407: [WebAssembly] Handle cleanuppad with no parent in Wasm SjLj

Thu Jan 27 14:28:12 PST 2022

aheejin marked an inline comment as done.
aheejin added inline comments.

================
Comment at: llvm/test/CodeGen/WebAssembly/lower-wasm-ehsjlj.ll:270
+; CHECK: %call13 = invoke {{.*}} %struct.Temp* @_ZN4TempD2Ev(%struct.Temp*
+; CHECK-NEXT:    to label {{.*}} unwind label %catch.dispatch.longjmp
+  cleanupret from %0 unwind to caller
----------------
dschuff wrote:
> is it catch.longjmp.dispatch or catch.dispatch.longjmp?
Oh it's `catch.dispatch.longjmp`: https://github.com/llvm/llvm-project/blob/e900f0584e904e884207e40c80a8126824ef4c41/llvm/lib/Target/WebAssembly/WebAssemblyLowerEmscriptenEHSjLj.cpp#L1702-L1703

Fixed the comments.

Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D118407