This is the mail archive of the binutils@sourceware.org mailing list for the binutils project.

Index Nav:	[Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav:	[Date Prev] [Date Next]	[Thread Prev] [Thread Next]
Other format:	[Raw text]

Re: broken testresults with "[PATCH] decoded output of .debug_line from readelf -wL"

From: Nick Clifton <nickc at redhat dot com>
To: Hans-Peter Nilsson <hp at bitrange dot com>, "H.J. Lu" <hjl dot tools at gmail dot com>
Cc: Torleif Sandnes <torleif dot sandnes at gmail dot com>, binutils at sourceware dot org
Date: Sat, 12 Apr 2008 08:58:03 +0100
Subject: Re: broken testresults with "[PATCH] decoded output of .debug_line from readelf -wL"
References: <dfb869080702250731l324dd289mdff9dc34ce4dcedd@mail.gmail.com> <45E426FA.8090401@redhat.com> <dfb869080804090251o2c678718hbe4f981a1a4c4c0b@mail.gmail.com> <c17be2b30804091557h2647d120o5589904bc72a4092@mail.gmail.com> <dfb869080804100247je3e16a7o6db3f4ef72c51567@mail.gmail.com> <20080410122410.GA16652@caradoc.them.org> <dfb869080804100556k2ff6d678w3552257f9df263c2@mail.gmail.com> <dfb869080804101346v2800909fq9a77d39619351c3d@mail.gmail.com> <47FF299C.5030609@redhat.com> <20080411131248.O24377@dair.pair.com> <6dc9ffc80804111020p1efa70dev5c628d47ae2b1652@mail.gmail.com> <20080411140220.X80610@dair.pair.com>

Hi H.J. Hi Hans-Peter

H.J.: Please do check your patch in.

> Hans-Peter Nilsson wrote:

But was that really intended?  Should -wl really
have been affected?  Not obvious to me.

No, it would have been better if -wl had not been affected, but I think that we might as well stick with the change. The new version is more informative (it tells the user that the output is not being interpreted), and since it is now in place we might as well keep it.

I apologize for not running the gas tests before committing the patch. I ran the linker and binutils testsuites, but did not think to check the assembler.

Cheers
  Nick

References:
- Re: [PATCH] decoded output of .debug_line from readelf -wL
  - From: Torleif Sandnes
- Re: [PATCH] decoded output of .debug_line from readelf -wL
  - From: Cary Coutant
- Re: [PATCH] decoded output of .debug_line from readelf -wL
  - From: Torleif Sandnes
- Re: [PATCH] decoded output of .debug_line from readelf -wL
  - From: Daniel Jacobowitz
- Re: [PATCH] decoded output of .debug_line from readelf -wL
  - From: Torleif Sandnes
- Re: [PATCH] decoded output of .debug_line from readelf -wL
  - From: Torleif Sandnes
- Re: [PATCH] decoded output of .debug_line from readelf -wL
  - From: Nick Clifton
- broken testresults with "[PATCH] decoded output of .debug_line from readelf -wL"
  - From: Hans-Peter Nilsson
- Re: broken testresults with "[PATCH] decoded output of .debug_line from readelf -wL"
  - From: H.J. Lu
- Re: broken testresults with "[PATCH] decoded output of .debug_line from readelf -wL"
  - From: Hans-Peter Nilsson

Index Nav:	[Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav:	[Date Prev] [Date Next]	[Thread Prev] [Thread Next]