Remaining arch issue in Newlib optimized ARM ASM memchr for Cortex-R
Richard Earnshaw
rearnsha@arm.com
Thu Jul 3 12:49:00 GMT 2014
On 03/07/14 13:17, Nicholas Clifton wrote:
> Hi Jeff,
>
> > Pavel Pisa wrote:
>> Index: newlib-2.1/newlib/libc/machine/arm/memchr.S
>> ===================================================================
>> --- newlib-2.1.orig/newlib/libc/machine/arm/memchr.S 2011-10-18 13:35:17.000000000 +0200
>> +++ newlib-2.1/newlib/libc/machine/arm/memchr.S 2014-07-03 00:38:47.598049234 +0200
>> @@ -46,7 +46,11 @@
>> @ tidy
>>
>> .syntax unified
>> +#ifdef __ARM_ARCH_7R__
>> + .arch armv7-r
>> +#else
>> .arch armv7-a
>> +#endif
>>
>> #include "arm_asm.h"
>
> This patch looks good to me.
>
> Cheers
> Nick
>
>
>
Not quite. It's better than what's already there, but misses the
ARMv6t2 case -- see a couple of lines later. I think a simple change to
.arch armv6t2
should handle all v7-a and v7-r cases.
R.
More information about the Newlib
mailing list