This is the mail archive of the
mailing list for the binutils project.
Re: [PATCH 04/12] bfd/ * bfd/elf32-microblaze.c: Correct adjustment of global symbols
Hi Mike, Michael, All
On 11 October 2012 04:29, Michael Eager <firstname.lastname@example.org> wrote:
> On 10/10/2012 11:05 AM, Mike Frysinger wrote:
>> On Tuesday 09 October 2012 22:13:57 email@example.com wrote:
> Bfd has very little debugging code, but there is an example in bfdwin.c.
> uses a global symbol which can be set by a debugger to enable debug output.
> better method is to enable this by a command line option, but this may not
> as practical for bfd, which is used by several components.
> If the debug code is really necessary, it should not require someone
> to modify the file to enable it.
> I think that this is development code and is not needed.
Yes, this is primarily debug information left over from development,
and I would prefer to remove it from the patch.
I'll update this in the resubmitted patch series.
> Michael Eager firstname.lastname@example.org
> 1960 Park Blvd., Palo Alto, CA 94306 650-325-8077