Re: [PATCH] Do not dump location for compare_debug

2012-11-26 Thread Richard Biener
On Thu, Nov 1, 2012 at 1:26 AM, Dehao Chen  wrote:
> Hi,
>
> When -fcompare_debug is used, what we really want to do is to compare
> instructions between the -g version and -gtoggle version. However,
> current dump file still contains the source line in its rtl dump. This
> patch changes to only dump rtl without dumping its source info.
>
> Bootstrapped and passed gcc regression test on x86_64.
>
> Is it okay for trunk?

I'm not sure - it at least "works" comparing with them.  Alex - was source
location info included on purpose here?

Thanks,
Richard.

> Thanks,
> Dehao
>
> 2012-10-31  Dehao Chen  
>
> * final.c (rest_of_clean_state): Dump the final rtl without location.


Re: [PATCH] Do not dump location for compare_debug

2012-11-08 Thread Dehao Chen
ping...

On Wed, Oct 31, 2012 at 5:26 PM, Dehao Chen  wrote:
> Hi,
>
> When -fcompare_debug is used, what we really want to do is to compare
> instructions between the -g version and -gtoggle version. However,
> current dump file still contains the source line in its rtl dump. This
> patch changes to only dump rtl without dumping its source info.
>
> Bootstrapped and passed gcc regression test on x86_64.
>
> Is it okay for trunk?
>
> Thanks,
> Dehao
>
> 2012-10-31  Dehao Chen  
>
> * final.c (rest_of_clean_state): Dump the final rtl without location.


[PATCH] Do not dump location for compare_debug

2012-10-31 Thread Dehao Chen
Hi,

When -fcompare_debug is used, what we really want to do is to compare
instructions between the -g version and -gtoggle version. However,
current dump file still contains the source line in its rtl dump. This
patch changes to only dump rtl without dumping its source info.

Bootstrapped and passed gcc regression test on x86_64.

Is it okay for trunk?

Thanks,
Dehao

2012-10-31  Dehao Chen  

* final.c (rest_of_clean_state): Dump the final rtl without location.


compare_debug.patch
Description: Binary data