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: [patch bfd]: Avoid to remove leading underscore for anon-object for pe-coff targets with empty USER_LABEL_PREFIX


On Fri, Sep 09, 2011 at 03:20:48PM +0200, Kai Tietz wrote:
>2011/9/9 Christopher Faylor <cgf-use-the-mailinglist-please@sourceware.org>:
>> On Fri, Sep 09, 2011 at 02:41:58PM +0200, Kai Tietz wrote:
>>>this patch avoids the removal of leading underscores for pe-coff
>>>targets with empty USER_LABEL_PREFIX (like x86_64-pe-coff, and arm).
>>>
>>>Regression-tested for i686-w64-mingw32 and x86_64-w64-mingw32. ?Ok for apply?
>>
>> Shouldn't that testing include a non-w64 Windows target too?
>>
>>>ChangeLog
>>>
>>>2011-09-09 ?Kai Tietz ?<ktietz@redhat.com>
>>>
>>> ? ? ? ?* peicode.h (pe_ILF_build_a_bfd): Don't remove leading underscore
>>> ? ? ? ?for targets without symbol_leading_char.
>>
>> It looks ok to me, but I would like confirmation that it works with
>> mingw32 and Cygwin.
>
>I tested iit for 686-w64-mingw32 target - as I wrote - and this is a
>32-bit i686 target.  So test is sufficiant as there is in binutils'
>bfd code no difference between i686-pc-mingw32, i686-pc-cygwin, and
>i686-w64.mingw32 targets in this area.

Ok, then please check in.

cgf


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