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

[Louis Dionne via libcxx-commits]

================
@@ -55,41 +108,101 @@ def parse_args():
     )
 
     format_group = parser.add_argument_group("Output Format")
-    # FIXME: I find these names very confusing, although I like the
-    # functionality.
     format_group.add_argument(
-        "-q", "--quiet", help="Suppress no error output", action="store_true"
+        "--test-output",
+        help="Control whether the executed commands and their outputs are printed after each test has executed (default off)",
+        choices=["off", "failed", "all"],
+        default="off",
+        action=TestOutputAction,
+    )
+    format_group.add_argument(
+        "--print-result-after",
+        help="Control which the executed test names and results are printed after each test has executed (default all)",
----------------
ldionne wrote:

Should you mention how this interacts with `--test-output`? In your RFC, you mention that `--test-output=X` requires `--print-result-after=<...>` that is at least as "strong" as `X`. It might be worth mentioning in the `--help` output.

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