This is the mail archive of the gcc-patches@gcc.gnu.org mailing list for the GCC 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 i386 win32]: Fix local symbols for x64


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.


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