[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