[PATCH] D108583: [WebAssembly] Use SSAUpdaterBulk in LowerEmscriptenSjLj

Heejin Ahn via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Mon Aug 23 13:50:51 PDT 2021


aheejin added inline comments.


================
Comment at: llvm/test/CodeGen/WebAssembly/lower-em-sjlj.ll:225
 ; CHECK: for.inc.split:
-  ; CHECK: %var[[VARNO]] = phi i32 [ %var, %for.inc ]
+  ; CHECK: %var[[VARNO]] = phi i32 [ undef, %if.end ], [ %var, %for.inc ]
 }
----------------
This is just an order change of `phi` clauses and doesn't mean the number of clause has been increased from 1 to 2. Previously we were only checking the first clause.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D108583



More information about the llvm-commits mailing list