[PATCH]: Pass -no_pie on SYSTEMSPEC for darwin11

IainS developer@sandoe-acoustics.co.uk
Sat Jun 18 08:57:00 GMT 2011


Hi Jack,

On 18 Jun 2011, at 03:57, Jack Howarth wrote:

> On Fri, Jun 17, 2011 at 12:04:34PM +0100, IainS wrote:
>> Hi Jack,
>> ===
>> ****
>> kludge to suppress pie for for dylibs (use in place of the darwin9.h
>> hunk from the attachment on PR49371).
>
> Iain,
>   This change should be unncessary since -pie on the linker only  
> operates
> on executables (hence the name PIE).

On darwin < 11 .. which don't default to linking pie:

As stated, it's a kludge to allow fpie to be put as a test-suite  
option...
... since the options are passed verbatim to library builds, all the  
dylib builds fail otherwise ...

if there is an alternate way of filtering 'fpie/fPIE' out of dylib  
builds, that could be more globally applied the to the test suite,  
that would be a good fix to make.

(the kludge is expedient to test the hypothesis that pie works with  
libjava on darwin 9 & 10 -- FAOD: I am not suggesting it for trunk)...

Iain

>> Index: gcc/config/darwin9.h
>> ===================================================================
>> --- gcc/config/darwin9.h        (revision 175110)
>> +++ gcc/config/darwin9.h        (working copy)
>> @@ -35,6 +35,12 @@ along with GCC; see the file COPYING3.  If not see
>> /* Tell collect2 to run dsymutil for us as necessary.  */
>> #define COLLECT_RUN_DSYMUTIL 1
>>
>> +#undef PIE_SPEC
>> +#define PIE_SPEC \
>> +  "%{fpie|pie|fPIE:  %{!Zdynamiclib: \
>> +     %{mdynamic-no-pic: %n'-mdynamic-no-pic' overrides '-pie', '- 
>> fpie'
>> or '-fPIE'; \
>> +      :-pie}}}"
>> +
>> #undef  ASM_OUTPUT_ALIGNED_COMMON
>> #define ASM_OUTPUT_ALIGNED_COMMON(FILE, NAME, SIZE,  
>> ALIGN)             \
>>   do  
>> {                                                                 \



More information about the Gcc-patches mailing list