[PATCH] D55956: [WebAssembly] Fix invalid machine instrs in -O0, verify in tests
Thomas Lively via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Thu Dec 20 14:55:57 PST 2018
tlively marked an inline comment as done.
tlively added inline comments.
================
Comment at: test/CodeGen/WebAssembly/simd-noopt.ll:9
+
+define i64 @foo(<2 x i64> %vec) #0 {
+entry:
----------------
aheejin wrote:
> tlively wrote:
> > aheejin wrote:
> > > No check lines?
> > I'm mostly just interested in this not crashing, but would be happy to add some if you think it would be better.
> Not crashing meaning, not crashing llvm regression test or V8? If you mean the latter, I guess it's better to check that v2i64 types are not generated by one or two `CHECK-NOT` lines.
Oh no, sorry, I meant not crashing LLVM. It was previously crashing because it was generating MachineInstrs that used a register that had never been defined.
Repository:
rL LLVM
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D55956/new/
https://reviews.llvm.org/D55956
More information about the llvm-commits
mailing list