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: [PowerPC] Fix setjmp test fails


On Sun, Oct 31, 2010 at 4:47 AM, Alan Modra <amodra@gmail.com> wrote:
> gcc.dg/stackalign/setup-{1,3,4}.c exposed two bugs in my -msecure-plt
> support for powerpc-linux. ?First, rs6000_pic_labelno can only be used
> when rs6000_emit_load_toc_table is called from the prologue. ?Using it
> again in the body of a function results in
> ?Error: symbol `.LCF1' is already defined.
> Fixed easily by using a normal label, rather than the special one for
> -mrelocatable. ?Secondly, load_toc_v4_PIC_3b had its constraints
> reversed, allowing gcc to choose r0 as the input to an addis
> instruction.
>
> Bootstrapped and regression tested powerpc-linux. ?OK to apply
> everywhere?
>
> ? ? ? ?* config/rs6000/rs6000.c (rs6000_pic_labelno): Make static.
> ? ? ? ?(rs6000_emit_load_toc_table): Don't use rs6000_pic_labelno when
> ? ? ? ?TARGET_SECURE_PLT.
> ? ? ? ?* config/rs6000/sysv4.h (rs6000_pic_labelno): Don't declare.
> ? ? ? ?* config/rs6000/rs6000.md (load_toc_v4_PIC_3b): Use "b" constraint
> ? ? ? ?on input, "r" on output.

Okay.

Thanks, David


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