[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