Re: PATCH: PR bootstrap/53472: contrib/compare-debug should strip out .comment section

2012-05-24 Thread Paolo Bonzini
> 2012-05-24  H.J. Lu   > >        PR bootstrap/53472 >        * contrib/compare-debug (remove_comment): New function. >        Remove any .comment sections if the first cmp failed. Ok. Paolo > diff --git a/contrib/compare-debug b/contrib/compare-debug > index 010d17f..fb8986d 100755 > --- a/con

Re: PATCH: PR bootstrap/53472: contrib/compare-debug should strip out .comment section

2012-05-24 Thread H.J. Lu
On Thu, May 24, 2012 at 2:43 PM, Jakub Jelinek wrote: > On Thu, May 24, 2012 at 02:32:59PM -0700, H.J. Lu wrote: >> This patch works on openSUSE 12.1.  OK to install? > > Can't you do that only if the first cmp failed? > In that case strip and cmp again... > Like this? OK to install? Thanks. -

Re: PATCH: PR bootstrap/53472: contrib/compare-debug should strip out .comment section

2012-05-24 Thread Jakub Jelinek
On Thu, May 24, 2012 at 02:32:59PM -0700, H.J. Lu wrote: > This patch works on openSUSE 12.1. OK to install? Can't you do that only if the first cmp failed? In that case strip and cmp again... > 2012-05-24 H.J. Lu > > PR bootstrap/53472 > * contrib/compare-debug (remove_comment):

Re: PATCH: PR bootstrap/53472: contrib/compare-debug should strip out .comment section

2012-05-24 Thread H.J. Lu
On Thu, May 24, 2012 at 9:53 AM, H.J. Lu wrote: > On Thu, May 24, 2012 at 7:43 AM, H.J. Lu wrote: >> On Thu, May 24, 2012 at 7:35 AM, Paolo Bonzini wrote: >>> Il 24/05/2012 16:01, H.J. Lu ha scritto: configure checks if contrib/compare-debug actually works for comparing a debug and non

Re: PATCH: PR bootstrap/53472: contrib/compare-debug should strip out .comment section

2012-05-24 Thread H.J. Lu
On Thu, May 24, 2012 at 7:43 AM, H.J. Lu wrote: > On Thu, May 24, 2012 at 7:35 AM, Paolo Bonzini wrote: >> Il 24/05/2012 16:01, H.J. Lu ha scritto: >>> configure checks if contrib/compare-debug actually works for comparing a >>> debug and non-debug .o file before enabling bootstrap-debug.  But so

Re: PATCH: PR bootstrap/53472: contrib/compare-debug should strip out .comment section

2012-05-24 Thread H.J. Lu
On Thu, May 24, 2012 at 7:35 AM, Paolo Bonzini wrote: > Il 24/05/2012 16:01, H.J. Lu ha scritto: >> configure checks if contrib/compare-debug actually works for comparing a >> debug and non-debug .o file before enabling bootstrap-debug.  But some >> compilers encode some command line options (amon

Re: PATCH: PR bootstrap/53472: contrib/compare-debug should strip out .comment section

2012-05-24 Thread Paolo Bonzini
Il 24/05/2012 16:01, H.J. Lu ha scritto: > configure checks if contrib/compare-debug actually works for comparing a > debug and non-debug .o file before enabling bootstrap-debug. But some > compilers encode some command line options (among them -g) into a special > .comment section. This patch re

PATCH: PR bootstrap/53472: contrib/compare-debug should strip out .comment section

2012-05-24 Thread H.J. Lu
Hi, configure checks if contrib/compare-debug actually works for comparing a debug and non-debug .o file before enabling bootstrap-debug. But some compilers encode some command line options (among them -g) into a special .comment section. This patch removes .comment section before comparing debu