[PATCH] D90281: [FileCheck] Report missing prefixes when more than one is provided.

Simon Pilgrim via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Tue Nov 10 07:51:41 PST 2020


RKSimon added a comment.

In D90281#2385772 <https://reviews.llvm.org/D90281#2385772>, @mtrofin wrote:

> lit.local.cfg would "blanket-allow" future tests, too. Would that be desirable?

Yes I'm finding that 99% of llvm tests that were failing were just poor check-prefix setup (usually copy+paste or too much emphasis on an unnecessarily complex prefix hierarchy) - so until you can flip the default value in FileCheck itself we should be able to use lit.local.cfg files in the subfolder root to keep them clean once they've been fixed up.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D90281



More information about the llvm-commits mailing list