This is the mail archive of the
gcc-patches@gcc.gnu.org
mailing list for the GCC project.
Re: PING^3: [PATCH]: New configure options that make the compiler use -fPIE and -pie as default option
- From: "H.J. Lu" <hjl dot tools at gmail dot com>
- To: Joseph Myers <joseph at codesourcery dot com>
- Cc: Magnus Granberg <zorry at gentoo dot org>, GCC Patches <gcc-patches at gcc dot gnu dot org>
- Date: Mon, 18 May 2015 18:25:13 -0700
- Subject: Re: PING^3: [PATCH]: New configure options that make the compiler use -fPIE and -pie as default option
- Authentication-results: sourceware.org; auth=none
- References: <CAMe9rOrw7OFObOmOCG1wM2Hfr_hpKo-wY0_17ngB89CTQ_34Mw at mail dot gmail dot com> <alpine dot DEB dot 2 dot 10 dot 1505072104300 dot 20024 at digraph dot polyomino dot org dot uk> <CAMe9rOrcADb1t2Ch0LDy9SOaS9RDoj4BGgo8qL+6jtg1tnLA8g at mail dot gmail dot com> <3072346 dot CTCrhcXNep at laptop1 dot gw dot ume dot nu> <alpine dot DEB dot 2 dot 10 dot 1505190109380 dot 16300 at digraph dot polyomino dot org dot uk>
On Mon, May 18, 2015 at 6:11 PM, Joseph Myers <joseph@codesourcery.com> wrote:
> On Tue, 19 May 2015, Magnus Granberg wrote:
>
>> > > <https://gcc.gnu.org/ml/gcc-patches/2015-03/msg00393.html> noted a
>> > > possible issue with MIPS. Actually, rather more config/*.h and
>> > > config/*/*.h headers contain specs testing for (-fpie, -fPIE, -fno-pie,
>> > > -fno-PIE, -pie) options, which would be affected by these changes. I'd
>> > > say this patch should include an initial attempt at adjusting those config
>> > > headers, which should be an essentially mechanical change not requiring
>> > > understanding anything target-specific. For link-time specs, that may
>> > > mean using PIE_SPEC and NO_PIE_SPEC. For compile-time specs, similar new
>> > > macros would be added. Given such adjustments included in the patch and
>> > > the relevant target maintainers CC:ed, I might then be inclined to approve
>> > > the patch on the basis of allowing a week for target maintainers to test
>> > > the changes for their targets before commit, as I don't see any major
>> > > problems with it beyond the need to update the target-specific specs.
>> >
>> > Here is the updated patch. I will post patches for cris, mips, powerpc
>> > and sparc separately. The target maintainers should be able to adjust
>> > backend ASM_SPEC with FPIE_OR_FPIC_SPEC and
>> > NO_FPIE_AND_FPIC_SPEC.
>> >
>> > OK for trunk?
>> >
>> > Thanks.
>> PIng
>>
>> Any progress on this?
>
> Have updates for all affected specs for all targets been posted? I just
> saw a small and apparently arbitrary subset of targets with patches, and
> no explanation of how those targets were identified or why the other
> targets with specs mentioning the options in question did not need
> updates.
>
I only posted patches for an arbitrary subset of targets because
1. Not everyone is interested in --enable-default-pie.
2. I can't tests all targets myself.
If patches for all targets is the only blocker before the patch
will be approved or target maintainers will help me test the patch,
I will post patches for each target affected.
--
H.J.