[PATCH v2] Use correct ARM architecture profile for optimized ARMv6t2/v7 memchr version.

Richard Earnshaw rearnsha@arm.com
Mon Jul 7 14:00:00 GMT 2014


On 07/07/14 14:47, Pavel Pisa wrote:
> This prevents next error at link time for Cortex-R multilib option
> 
> Conflicting architecture profiles A/R
> .../arm-rtems4.11/bin/ld: failed to merge target
> 
> Use solution proposed by Richard Earnshaw.
> ".arch armv6t2" defines null profile which should be
> be upward compatible with all targets using optimized
> version.
> 

Thanks.

I've tweaked the patch slightly to conform with general coding
conventions and committed the following:

R.

2014-07-07  Pavel Pisa  <pisa@cmp.felk.cvut.cz>
	    Richard Earnshaw  <rearnsha@arm.com>

	* libc/machine/arm/memchr.S (.arch): Require revision ARMv6t2.
-------------- next part --------------
Index: libc/machine/arm/memchr.S
===================================================================
RCS file: /cvs/src/src/newlib/libc/machine/arm/memchr.S,v
retrieving revision 1.1
diff -u -r1.1 memchr.S
--- libc/machine/arm/memchr.S	18 Oct 2011 11:35:17 -0000	1.1
+++ libc/machine/arm/memchr.S	7 Jul 2014 13:57:42 -0000
@@ -45,8 +45,10 @@
 @    Make conditional on CPU type
 @    tidy
 
+@ This code requires armv6t2 or later.  Uses Thumb2.
+
 	.syntax unified
-	.arch armv7-a
+	.arch armv6t2
 
 #include "arm_asm.h"
 


More information about the Newlib mailing list