[PATCH] D154984: [lit] Drop "Script:", make -v and -a imply -vv

Fangrui Song via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Fri Jul 14 13:37:44 PDT 2023


MaskRay added a comment.

Thanks for the clean up. I think the new option (-a -v) semantics are more orthogonal. I always use `-a` or `-vv` and avoid `-v`.

My understanding of the behavior of `$build/bin/llvm-lit -a test`: for a succeeded test, all commands will be printed. For a failed test, all commands til the failed one will be printed.

I wonder whether we can change the `Command Output (stderr):` (pipe `a|b` is printed as `+ a\n| b`; the plus sign seems to get in the way for copy pasting) to be like `Script:` (`a|b` is printed as is).
Perhaps it will fulfill @ldionne's request "This is non-blocking, but I do use Script: a lot to reproduce libc++ test failures. Here's an example:"


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D154984



More information about the llvm-commits mailing list