[PATCH] D90281: [FileCheck] Report missing prefixes when more than one is provided.
Mircea Trofin via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Wed Oct 28 18:23:54 PDT 2020
mtrofin added a comment.
In D90281#2360025 <https://reviews.llvm.org/D90281#2360025>, @MaskRay wrote:
> You can add the option in this patch and create a RFC on llvm-dev (I think people have favored the idea in the past but I cannot recall the thread now). Maintainers do one of the two:
>
> - Add --allow-unused-prefixes if their use cases are similar to @jdoerfert and @jdenny's
> - Fix the test prefixes (usually due to stale RUN lines after some check prefixes related changes). I think the majority of ~1500 tests belongs to this category.
>
> This can be a collaborative spreadsheet so that people can claim the work.
SGTM - I'll post a RFC tomorrow, with list. This will also give a chance to folks in other time zones to chime in, if there are some additional considerations (less churn to the RFC, if that is the case)
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