[PATCH] D79324: [WebAssembly] Fix block marker placing after fixUnwindMismatches

Heejin Ahn via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Mon May 4 05:50:07 PDT 2020


aheejin marked 4 inline comments as done.
aheejin added inline comments.


================
Comment at: llvm/lib/Target/WebAssembly/WebAssemblyCFGStackify.cpp:292
+#endif
+    }
 
----------------
Fix for bug 1


================
Comment at: llvm/lib/Target/WebAssembly/WebAssemblyCFGStackify.cpp:1090
       BuildMI(EHPadLayoutPred, DL, TII.get(WebAssembly::BR)).addMBB(Cont);
+      BrDests.push_back(Cont);
     }
----------------
Fix for bug 4


================
Comment at: llvm/lib/Target/WebAssembly/WebAssemblyCFGStackify.cpp:1203
+          EHPad->removeSuccessor(BrDest);
+      }
       MBB->addSuccessor(NestedEHPad);
----------------
Fix for bug 2


================
Comment at: llvm/lib/Target/WebAssembly/WebAssemblyCFGStackify.cpp:1241
+  if (AppendixBB)
+    BrDests.push_back(AppendixBB);
+
----------------
Fix for bug 3


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D79324





More information about the llvm-commits mailing list