[llvm] [AMDGPU][test]added remove duplicate options for update_mc_test_check script (PR #111769)

Brox Chen via llvm-commits llvm-commits at lists.llvm.org
Fri Oct 11 15:00:54 PDT 2024


================
@@ -118,6 +119,19 @@ def main():
         default=None,
         help="Set a default -march for when neither triple nor arch are found in a RUN line",
     )
+    parser.add_argument(
+        "--remove-duplicate",
+        action="store_true",
+        default=False,
+        help="remove duplicated test line if found",
+    )
+    parser.add_argument(
+        "--sort",
+        action="store_true",
+        default=False,
----------------
broxigarchen wrote:

The same reason as above. I think it's better asking user to run --sort manually.

I think we can add a hint by checking the auto-generated header line
`# auto updated by update_mc_test_check.py UTC_ARGS: --sort`

if there is a `--sort` there but there is no `--sort` being passed by the arguments, we can shot a warning

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


More information about the llvm-commits mailing list