[libcxx-commits] [compiler-rt] [libcxx] [llvm] [utils] revamp options controlling lit's output (PR #167192)

[Louis Dionne via libcxx-commits]

https://github.com/ldionne approved this pull request.

I am a big fan of your RFC and this PR. I've always found Lit's verbosity options to be confusing, and IMO this makes them a lot easier to work with. I'd almost be in favour of dropping the old options, but you're hugely decreasing the burden of this PR by keeping backwards compatibility, so I think that's the smart thing to do.

I have some comments, but this basically LGTM. I pulled your PR and ran the libc++ test suite with your patch applied, and my usual workflow was working as expected.

You probably want to have other people's feedback as well, but from my side this is 👍🏻.

https://github.com/llvm/llvm-project/pull/167192