[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