[llvm] [WebAssembly] Fix phi handling for Wasm SjLj (PR #99730)

Heejin Ahn via llvm-commits llvm-commits at lists.llvm.org
Fri Jul 19 18:54:35 PDT 2024


================
@@ -777,6 +777,8 @@ void WebAssemblyLowerEmscriptenEHSjLj::rebuildSSA(Function &F) {
   SSAUpdaterBulk SSA;
   for (BasicBlock &BB : F) {
     for (Instruction &I : BB) {
+      if (I.getType()->isVoidTy())
+        continue;
----------------
aheejin wrote:

This is not related to this PR and a NFC drive-by small fix. Because we are adding defs, void type instructions don't have any effects at all when added to `SSAUpdaterBulk` anyway.

https://github.com/llvm/llvm-project/pull/99730


More information about the llvm-commits mailing list