Christian Sayer
2009-Jan-30 09:55 UTC
[LLVMdev] [patch] color nodes matching a string pattern when displaying graphs
Hello, the patch attatched adds the option -view-dag-color-node=<list> to llc. The list of strings is matched with the names of the labels of nodes when displaying a graph, so that matching nodes are displayed in a colored bold style. I find this extremely usful on selectionDAGs to spot specific nodes in larger graphs. You may complain that the implementation should be interacting more with the DOTGraphTraits class, but OTOH this might be an interesting feature for any type of graph, and as such it is certainly the lightest implementation. Anyway, if this is considered an interesting feature for the mainline but unacceptable this way please let me know, I might eventually change it according to your suggestions. Regards, Christian -- please ignore: CONFIDENTIAL NOTICE: The contents of this message, including any attachments, are confidential and are intended solely for the use of the person or entity to whom the message was addressed. If you are not the intended recipient of this message, please be advised that any dissemination, distribution, or use of the contents of this message is strictly prohibited. If you received this message in error, please notify the sender. Please also permanently delete all copies of the original message and any attached documentation. Thank you. -------------- next part -------------- A non-text attachment was scrubbed... Name: GraphWriter_color.patch Type: application/octet-stream Size: 2235 bytes Desc: GraphWriter_color.patch URL: <http://lists.llvm.org/pipermail/llvm-dev/attachments/20090130/452379cd/attachment.obj>
Christian Sayer
2009-Jan-30 10:55 UTC
[LLVMdev] [patch] color nodes matching a string pattern when displaying graphs
BTW, I noticed that the Support directory is not part of the PARALLEL_DIRS in lib/Makefile, so it is not recompiled when doing 'make' in lib/ although the other 'parallel dirs' are users of Support. Is this intentional?> Anyway, if this is considered an interesting feature for the > mainline but unacceptable this way please let me know, I > might eventually change it according to your suggestions.Don't blame me on this, I just meant don't count on me to do it within he day :) Regards, Christian -- please ignore: CONFIDENTIAL NOTICE: The contents of this message, including any attachments, are confidential and are intended solely for the use of the person or entity to whom the message was addressed. If you are not the intended recipient of this message, please be advised that any dissemination, distribution, or use of the contents of this message is strictly prohibited. If you received this message in error, please notify the sender. Please also permanently delete all copies of the original message and any attached documentation. Thank you.