This is the mail archive of the
gcc-patches@gcc.gnu.org
mailing list for the GCC project.
Re: [PATCH] Add CFI entries for ARM Linux idiv0 / ldiv0
- From: Ramana Radhakrishnan <ramana dot gcc at googlemail dot com>
- To: James Lemke <jwlemke at codesourcery dot com>
- Cc: Ramana Radhakrishnan <ramana dot radhakrishnan at foss dot arm dot com>, gcc-patches <gcc-patches at gcc dot gnu dot org>
- Date: Tue, 23 Jun 2015 17:22:31 +0100
- Subject: Re: [PATCH] Add CFI entries for ARM Linux idiv0 / ldiv0
- Authentication-results: sourceware.org; auth=none
- References: <5580945D dot 4060608 at codesourcery dot com> <55897421 dot 5020602 at foss dot arm dot com> <558986EE dot 7020402 at codesourcery dot com>
- Reply-to: ramrad01 at arm dot com
On Tue, Jun 23, 2015 at 5:18 PM, James Lemke <jwlemke@codesourcery.com> wrote:
>
>>> Tested on gcc-trunk for arm-none-linux-gnueabi.
>>>
>>> OK to commit?
>>>>
>>>>
>>> 2015-06-16 James Lemke <jwlemke@codesourcery.com>
>>>
>>> libgcc/config/arm/
>>> * lib1funcs.S (aeabi_idiv0, aeabi_ldiv0): Add CFI entries for
>>> Linux ARM_EABI.
>>
>>
>> s/for Linux ARM EABI//
>>
>> given you handle both __ARM_EABI__ and the not __ARM_EABI__ targets in
>> the source.
>>
>> This is OK if no regressions.
>
>
> I saw no regressions for arm-none-linux-gnueabi.
> However, I don't have access to a non-eabi linux target.
> Shall I commit with the non-eabi portions or remove them?
> Jim.
I have no access to a non-EABI arm target to test this, not sure if
there are any intree anymore (probably vxworks ?) . It looks sane on
a read - just apply it and look after regressions if anything gets
reported in bugzilla.
regards
Ramana
>
>
> --
> Jim Lemke, GNU Tools Sourcerer
> Mentor Graphics / CodeSourcery
> Orillia, Ontario