[PATCH] D66729: [WebAssembly] Fix SSA rebuilding in SjLj transformation

Heejin Ahn via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Mon Aug 26 14:47:05 PDT 2019


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


================
Comment at: llvm/test/CodeGen/WebAssembly/lower-em-sjlj.ll:245
+
+; CHECK: for.inc.split:
+  ; CHECK: %var[[VARNO]] = phi i32 [ %var, %for.inc ]
----------------
dschuff wrote:
> Should this be uncommented or removed?
Oh no, after the transformation, `for.inc` itself will be split into multiple BBs, so the `CHECK` line below will be in `for.inc.split` BB.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D66729





More information about the llvm-commits mailing list