[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