RFC: LRA for x86/x86-64 [2/9]

Vladimir Makarov vmakarov@redhat.com
Sat Sep 29 00:23:00 GMT 2012


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.



More information about the Gcc-patches mailing list