================ @@ -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. ---------------- jansvoboda11 wrote:
Hmm, I wanted to seed the random shuffle with something "stable" (like the repo revision number) to be able to diff in tests. I also wanted to add a `clang-scan-deps` flag to disable it entirely for development purposes. Is that something that you still consider a bad idea? https://github.com/llvm/llvm-project/pull/114079 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits