[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