[PATCH] D63582: [LICM & MSSA] Limit unsafe sinking and hoisting.

Alex Langford via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Thu Jun 20 16:52:56 PDT 2019


xiaobai added a comment.

The test `pr42294.ll` breaks when running the test suite on a build without asserts. Specifically, `opt` doesn't know about the option `-debug-only`. I confirmed that adding a `REQUIRES: asserts` to the beginning of the test fixes things. I'm not too familiar with this area, so if you have a way to make it work with non-asserts build then that would be great. Otherwise, we can just require that asserts are enabled for this test to run.


Repository:
  rL LLVM

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D63582/new/

https://reviews.llvm.org/D63582





More information about the llvm-commits mailing list