This is the mail archive of the
gcc-patches@gcc.gnu.org
mailing list for the GCC project.
Re: [patch i386 win32]: Fix local symbols for x64
- From: Kai Tietz <ktietz70 at googlemail dot com>
- To: Uros Bizjak <ubizjak at gmail dot com>
- Cc: Kai Tietz <Kai dot Tietz at onevision dot com>, GCC Patches <gcc-patches at gcc dot gnu dot org>
- Date: Mon, 7 Jun 2010 15:19:35 +0200
- Subject: Re: [patch i386 win32]: Fix local symbols for x64
- References: <OF09248114.88550100-ONC1257734.0043B0CC-C1257734.0044577D@onevision.de> <AANLkTimBctuFhAC5bWiGftedtbBJRXRFJ7PB0QOt2p6l@mail.gmail.com>
2010/5/31 Uros Bizjak <ubizjak@gmail.com>:
> On Mon, May 31, 2010 at 2:26 PM, Kai Tietz <Kai.Tietz@onevision.com> wrote:
>
>> This patch makes sure, that for x64 non-underscore target the local
>> symbols having prefix ".L" to avoid collisions with user symbols.
>> Testcases that have shown this issue are c-c++-common/raw-string-1.c and
>> c-c++-common/raw-string-2.c . Additional this patch fixes some testcases
>> in gcc.dg for x64 target.
>>
>>
>> ChangeLog
>>
>> gcc/
>> 2010-05-31 ?Kai Tietz
>>
>> ? ? ? ?* config/i386.c (ix86_output_addr_vec_elt): Make LPREFIX
>> ? ? ? ?argument for fprintf.
>> ? ? ? ?(ix86_output_addr_diff_elt): Likewise.
>> ? ? ? ?(x86_function_profiler): Likewise.
>> ? ? ? ?* config/cygming.h (LOCAL_LABEL_PREFIX): Fix
>> ? ? ? ?for x64 no-underscore.
>> ? ? ? ?(LPREFIX): Likewise.
>> ? ? ? ?(ASM_GENERATE_INTERNAL_LABEL): Likewise.
>>
>>
>> gcc/testsuite
>> 2010-05-31 ?Kai Tietz
>>
>> ? ? ? ?* gcc.dg/compound-literal-1.c: Fix for llp64.
>> ? ? ? ?* gcc.dg/pr32370.c: Likewise.
>> ? ? ? ?* gcc.dg/pr37561.c: Likewise.
>> ? ? ? ?* gcc.dg/pr41340.c: Likewise.
>> ? ? ? ?* gcc.dg/pr41551.c: Likewise.
>>
>> Tested for x86_64-w64-mingw32. Regression tested for i686-pc-cygwin and
>> i686-pc-mingw32. Ok for apply to trunk and to 4.5.x branch?
>
> x86 part is OK for trunk and after some time for 4.5.
>
> You still need approbal for testsuite...
>
> Thanks,
> Uros.
>
Back-merged from mainline to 4.5 branch at revision 160375.
Kai
--
| (\_/) This is Bunny. Copy and paste
| (='.'=) Bunny into your signature to help
| (")_(") him gain world domination