[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