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

Ivan Kosarev via llvm-commits llvm-commits at lists.llvm.org
Fri Oct 11 03:18:26 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,
----------------
kosarev wrote:

I guess not having this enabled by default might be problematic. Imagine some test cases added without sorting at random positions in one patch and then sorting them in another patch as some more test cases are added. This would also move cases added in the first patch. (I assume the idea is that ultimately we want keep all MC test files sorted at all times.)

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


More information about the llvm-commits mailing list