[PATCH] D39819: [cfi-verify] Add DOT graph printing for GraphResult objects.
Peter Collingbourne via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Fri Nov 10 16:44:31 PST 2017
pcc added inline comments.
================
Comment at: test/tools/llvm-cfi-verify/X86/dot-printing.s:12
+# Note: We check this behaviour using regexes so that the output doesn't depend
+# on the order of iteration of the classes used to implement the graphs. The
+# order of output in DOT format does not change the meaning of the graph.
----------------
Can you make the code deterministic by sorting the keys of `IntermediateNodes` before enumerating them? Then you can be more explicit in this test.
https://reviews.llvm.org/D39819
More information about the llvm-commits
mailing list