[llvm] [llvm][llvm-lit] Add option to create unique result file names if results already exist (PR #112729)

David Spickett via llvm-commits llvm-commits at lists.llvm.org
Thu Oct 17 08:33:27 PDT 2024


================
@@ -332,16 +339,21 @@ def parse_args():
     else:
         opts.shard = None
 
-    opts.reports = filter(
-        None,
-        [
-            opts.output,
-            opts.xunit_xml_output,
-            opts.resultdb_output,
-            opts.time_trace_output,
-        ],
+    opts.reports = list(
+        filter(
+            None,
+            [
+                opts.output,
+                opts.xunit_xml_output,
+                opts.resultdb_output,
+                opts.time_trace_output,
+            ],
+        )
     )
 
+    for report in opts.reports:
+        report.use_unique_output_file_name = opts.use_unique_output_file_name
----------------
DavidSpickett wrote:

Currently the argparse magic passes the value of xunit-xml-report to the type assigned to it, I couldn't find a built in way to pass another argument to the constructor of that type.

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


More information about the llvm-commits mailing list