[PATCH] D94853: [NFC] Disallow unused prefixes under Other

[Mircea Trofin via Phabricator via llvm-commits]

mtrofin marked 2 inline comments as done.
mtrofin added inline comments.


================
Comment at: llvm/test/Other/new-pass-manager.ll:369
 ; CHECK-EXT-NEXT: Finished llvm::Function pass manager run.
+; We don't have checks for CHECK-NOEXT here, but this simplifies the test, while
+; avoiding FileCheck complaining about the unused prefix.
----------------
aeubanks wrote:
> MaskRay wrote:
> > mtrofin wrote:
> > > @aeubanks : this test has CHECK-EXT asserts, but not CHECK-NOEXT. Is there a meaningful CHECK-NOEXT I could add? (other than {{.*}} )
> > `llvm/test/lit.cfg.py` defines `%llvmcheckext`, which expands to either CHECK-EXT or CHECK-NOEXT. You'll need two builds to verify.
> There's nothing to check for CHECK-NOEXT
(combining the 2 comments) so then adding the artificial CHECK-NOEXT check is ok? The alternative is to not use %llvmcheckext here, and just explicitly use the CHECK-EXT prefix.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D94853