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

George Rimar via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Wed Oct 28 00:37:59 PDT 2020


grimar added a comment.

In D90281#2358188 <https://reviews.llvm.org/D90281#2358188>, @jdoerfert wrote:

> In D90281#2358184 <https://reviews.llvm.org/D90281#2358184>, @mtrofin wrote:
>
>> [...]
>> If there were a (more terse) "--check-at-least-one-of-the-prefixes" with today's semantics, I assume the test generator produce tests using that option instead?
>
> The run lines are written/copied by us. We could easily switch to a new option like that.

I think we could add an option like `--allow-unused-prefixes`. It is probably sounds a bit simpler and can easily be added to generator I think.


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