[PATCH] D65121: [FileCheck][lit] Make FILECHECK_OPTS useful for their test suites

[Paul Robinson via Phabricator via llvm-commits]

probinson added a comment.

In D65121#1597516 <https://reviews.llvm.org/D65121#1597516>, @jdenny wrote:

> In D65121#1597486 <https://reviews.llvm.org/D65121#1597486>, @probinson wrote:
>
> > The lit part seems like a good thing.  It's unfortunate that lit/test/lit.cfg needs to know about FileCheck environment variables, but them's the breaks.
>
>
> If this issue were present in more test suites, I'd be tempted to extend FileCheck with a command-line option to encapsulate this use case in some manner.  As things are, that doesn't seem worth the trouble to me, but maybe others feel differently.


Then every test that had this issue would have to be modified to add the option.  I think it would have to be an environment variable that suppresses the other environment variables: `FILECHECK_IGNORE_ENV=1` or something. Then test suites could add that in their config files, and tests would be written as normal.

Luckily, so far we don't seem to need that level of confusion.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D65121