[llvm] b54e919 - [MLGO] Fix make_corpus_script.test

Aiden Grossman via llvm-commits llvm-commits at lists.llvm.org
Sun Jan 21 15:21:55 PST 2024


Author: Aiden Grossman
Date: 2024-01-21T23:20:56Z
New Revision: b54e919573a8ee03b4b813ad9705b36cba62232d

URL: https://github.com/llvm/llvm-project/commit/b54e919573a8ee03b4b813ad9705b36cba62232d
DIFF: https://github.com/llvm/llvm-project/commit/b54e919573a8ee03b4b813ad9705b36cba62232d.diff

LOG: [MLGO] Fix make_corpus_script.test

This test is currently failing as the order that the files end up in the
corpus description is somewhat dependent upon platform and the check is
checking for a specific order. This patch switches to using the
CHECK-DAG directive to make the checks order invariant to fix the broken
bots.

Added: 
    

Modified: 
    llvm/utils/mlgo-utils/tests/corpus/make_corpus_script.test

Removed: 
    


################################################################################
diff  --git a/llvm/utils/mlgo-utils/tests/corpus/make_corpus_script.test b/llvm/utils/mlgo-utils/tests/corpus/make_corpus_script.test
index a08780055f31f1..9b4d57330f0101 100644
--- a/llvm/utils/mlgo-utils/tests/corpus/make_corpus_script.test
+++ b/llvm/utils/mlgo-utils/tests/corpus/make_corpus_script.test
@@ -14,9 +14,9 @@
 ## Check that we get the expected command in the global command override
 # CHECK: "-test"
 # CHECK: "has_thinlto": false
-## Check that the modules are in the corpus description
-# CHECK: "test1"
-# CHECK: "test2"
+## Check that the modules are in the corpus description (order invariant)
+# CHECK-DAG: "test1"
+# CHECK-DAG: "test2"
 
 # RUN: ls %t.out.dir | FileCheck %s --check-prefix CHECK-DIR
 


        


More information about the llvm-commits mailing list