On 12-09-28 11:36 AM, Jeff Law wrote:
On 09/28/2012 09:21 AM, Vladimir Makarov wrote:
On 12-09-28 4:43 AM, Steven Bosscher wrote:
I have patches in the works to use the slim RTL dumping format more,
too, and to use the pretty-printer code so that printing strings with
escaped characters can be made more transparent (e.g. for use in
GraphViz dumps).
That would be nice. Slim printing is very useful for LRA which prints a
lot of changes in RTL code during all its work. Regular printing is
unreadable because of its volume. For LRA debugging I usually find a
suspicious place in slim dump and if I need more info I use regular dump
of the suspicious insn.
Perhaps it's time to rename sched-vis.c to print-rtl-slim.c? :-)
Yes, the name sched-vis.c is very misleading.
It seems to me this change ought to go forward now (rename to
print-rtl-slim.c and add print_value_slim.
WRT print_value_slim we have this block comment:
+/* Prints rtxes, I customarily classified as values. They're
+ constants, registers, labels, symbols and memory accesses. Print
+ them to file F. */
That block comment just doesn't make sense when I read it. It seems
like.
/* Print X, an RTL value node, to file F in slim format. Include
additional information if VERBOSE is nonzero.
Value nodes are constants, registers, labels, symbols and
memory. */
With that change I think you could make the obvious changes necessary
to rename to print-rtl-slim and check this patch in.
Thanks, Jeff. I will do it on Monday.
