[clang] [clang][deps] Stop sorting file dependencies (PR #114079)

Ben Langmuir via cfe-commits cfe-commits at lists.llvm.org
Tue Oct 29 09:33:51 PDT 2024


================
@@ -472,7 +465,8 @@ class FullDeps {
             JOS.attributeArray("command-line",
                                toJSONStrings(JOS, MD.getBuildArguments()));
             JOS.attribute("context-hash", StringRef(MD.ID.ContextHash));
-            JOS.attributeArray("file-deps", toJSONSorted(JOS, MD.FileDeps));
+            // TODO: Shuffle these to prevent tests from depending on the order.
----------------
benlangmuir wrote:

If the order is deterministic, why wouldn't we want to depend on it in tests?

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


More information about the cfe-commits mailing list