[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