[PATCH] D54411: [Codegen] Merge tail blocks with no successors after block placement
Heejin Ahn via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Sun Mar 24 09:27:57 PDT 2019
aheejin added inline comments.
Herald added a project: LLVM.
================
Comment at: test/CodeGen/WebAssembly/cfg-stackify.ll:544
; OPT: br_if 0, $pop{{[0-9]+}}{{$}}
-; OPT: br_if 1, $pop{{[0-9]+}}{{$}}
+; OPT: i32.ne
; OPT-NEXT: .LBB13_5:
----------------
aheejin wrote:
> You can probably drop this line, because this test is only checking control flow instructions.
I'm not sure if you are still planning to land this, but we deleted all `OPT` check lines in this file in D58953, so I think you should drop all changes to this file.
Repository:
rL LLVM
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D54411/new/
https://reviews.llvm.org/D54411
More information about the llvm-commits
mailing list