This is the mail archive of the
gcc-patches@gcc.gnu.org
mailing list for the GCC project.
Re: [AArch64][2/2] Implement -fpic for -mcmodel=small
- From: Jiong Wang <jiong dot wang at arm dot com>
- To: Christophe Lyon <christophe dot lyon at linaro dot org>
- Cc: Andreas Schwab <schwab at linux-m68k dot org>, Marcus Shawcroft <marcus dot shawcroft at gmail dot com>, gcc-patches <gcc-patches at gnu dot org>
- Date: Mon, 29 Jun 2015 10:19:55 +0100
- Subject: Re: [AArch64][2/2] Implement -fpic for -mcmodel=small
- Authentication-results: sourceware.org; auth=none
- References: <n99mw0zwmz7 dot fsf at arm dot com> <CAFqB+PwwScGMZdnHK3_xmkV4LCVqK_=WkhP3Nq_wR9cz55pqYA at mail dot gmail dot com> <n99h9py1sep dot fsf at arm dot com> <CAFqB+PxAdgcFJO34d7PQqgjGhUOcVHxwwqKi7D9OZAUNZQYtrA at mail dot gmail dot com> <n99fv5e24uc dot fsf at arm dot com> <n99egky24n9 dot fsf at arm dot com> <m2k2upk1tv dot fsf at linux-m68k dot org> <n998ub5s9ho dot fsf at arm dot com> <87r3oxv11a dot fsf at igel dot home> <n99h9pt9urb dot fsf at arm dot com> <CAKdteObsNxbKF2-NQw1nT5qZD5HWRkA24Wmewzq5idQ7xmCUCA at mail dot gmail dot com>
Christophe Lyon writes:
> On 27 June 2015 at 14:49, Jiong Wang <jiong.wang@arm.com> wrote:
>>
>> Andreas Schwab writes:
>>
>>> Jiong Wang <jiong.wang@arm.com> writes:
>>>
>>>> Andreas Schwab writes:
>>>>
>>>>> spawn -ignore SIGHUP /opt/gcc/gcc-20150627/Build/gcc/xgcc -B/opt/gcc/gcc-20150627/Build/gcc/ -fno-diagnostics-show-caret -fdiagnostics-color=never -O0 -w -fpie -c -o pr65593.o /opt/gcc/gcc-20150627/gcc/testsuite/gcc.c-torture/compile/pr65593.c
>>>>> /tmp/cc0Pymaf.s: Assembler messages:
>>>>> /tmp/cc0Pymaf.s:11: Error: unknown relocation modifier at operand 2 -- `ldr x0,[x0,#:gotpage_lo15:a]'
>>>>> /tmp/cc0Pymaf.s:19: Error: unknown relocation modifier at operand 2 --
>>>>> `ldr x2,[x0,#:gotpage_lo15:bar]'
>>>>
>>>> Andreas,
>>>>
>>>> The binutils patch has been upstreamed already. Please checkout the
>>>> latest binutils code.
>>>
>>> That must work with the current binutils.
>>
>> I see, agree.
>>
>> Will adopt current ILP32 feature detection mechanism into -fpic.
>>
>> And some new TLS patches may need similiar check also.
>>
>> Thanks.
>>
> Hi Jiong,
>
> I'm not sure I fully understand your answer. I believe you are going
> to change the testcase, right?
Sorry about the trouble.
I am going to add gcc_GAS_CHECK_FEATURE to detect whether those relocs
needed by -fpic supported. Just like what's done for ILP32 detection.
commit 891f2090d994d420cb56942e486f6bba3ede265f
Author: clyon <clyon@138bc75d-0d04-0410-961f-82ee72b054a4>
Date: Wed Dec 11 12:57:08 2013 +0000
>
> I'm seeing whole toolchain build failures (when building glibc) since
> you committed this, because of the new relocations. I understand your
> suggestion is to move to trunk binutils to have support for these new
> relocations. Any other things I could do to avoid changing the
> binutils version I use?
>
> Thanks
> Christophe.
>
>
>> --
>> Regards,
>> Jiong
>>
--
Regards,
Jiong