On Tue, Apr 3, 2012 at 12:47 PM, Mark Hatle <[email protected]> wrote: > A comment in the binutils sources indicate that it should support all of the > -march= parameters that gcc supports. The tune validations noted that > gas failed on -march=armv5e. > > It is not yet clear to me if this patch belongs upstream or not.
We should fix gcc instead so it emits -march=armv5te if gcc was given -march=armv5e when calling out assembler. -marmv5e is deprecated in gas. This wont fly > > Signed-off-by: Mark Hatle <[email protected]> > --- > .../binutils/binutils/binutils-armv5e.patch | 20 > ++++++++++++++++++++ > meta/recipes-devtools/binutils/binutils_2.22.bb | 3 ++- > 2 files changed, 22 insertions(+), 1 deletions(-) > create mode 100644 > meta/recipes-devtools/binutils/binutils/binutils-armv5e.patch > > diff --git a/meta/recipes-devtools/binutils/binutils/binutils-armv5e.patch > b/meta/recipes-devtools/binutils/binutils/binutils-armv5e.patch > new file mode 100644 > index 0000000..4ac1a9c > --- /dev/null > +++ b/meta/recipes-devtools/binutils/binutils/binutils-armv5e.patch > @@ -0,0 +1,20 @@ > +Add the armv5e architecture to binutils > + > +Binutils has a comment that indicates it is supposed to match gcc for all of > +the support "-march=" settings, but it was lacking the armv5e setting. This > +was a simple way to add it, as thumb instructions shouldn't be generated by > +the compiler anyway. > + > +Signed-off-by: Mark Hatle <[email protected]> > + > +diff -urN binutils-2.22.orig/gas/config/tc-arm.c > binutils-2.22/gas/config/tc-arm.c > +--- binutils-2.22.orig/gas/config/tc-arm.c 2011-11-21 03:29:31.000000000 > -0600 > ++++ binutils-2.22/gas/config/tc-arm.c 2012-04-03 12:07:42.230158760 -0500 > +@@ -22990,6 +22990,7 @@ > + {"armv4t", ARM_ARCH_V4T, FPU_ARCH_FPA}, > + {"armv4txm", ARM_ARCH_V4TxM, FPU_ARCH_FPA}, > + {"armv5", ARM_ARCH_V5, FPU_ARCH_VFP}, > ++ {"armv5e", ARM_ARCH_V5TE, FPU_ARCH_VFP}, > + {"armv5t", ARM_ARCH_V5T, FPU_ARCH_VFP}, > + {"armv5txm", ARM_ARCH_V5TxM, FPU_ARCH_VFP}, > + {"armv5te", ARM_ARCH_V5TE, FPU_ARCH_VFP}, > diff --git a/meta/recipes-devtools/binutils/binutils_2.22.bb > b/meta/recipes-devtools/binutils/binutils_2.22.bb > index 8b058fe..d6d19a1 100644 > --- a/meta/recipes-devtools/binutils/binutils_2.22.bb > +++ b/meta/recipes-devtools/binutils/binutils_2.22.bb > @@ -1,6 +1,6 @@ > require binutils.inc > > -PR = "r5" > +PR = "r6" > > LIC_FILES_CHKSUM="\ > file://src-release;endline=17;md5=4830a9ef968f3b18dd5e9f2c00db2d35\ > @@ -30,6 +30,7 @@ SRC_URI = "\ > file://libtool-rpath-fix.patch \ > file://clone-shadow.patch \ > file://binutils-powerpc-e5500.patch \ > + file://binutils-armv5e.patch \ > " > > SRC_URI[md5sum] = "ee0f10756c84979622b992a4a61ea3f5" > -- > 1.7.1 > > > _______________________________________________ > Openembedded-core mailing list > [email protected] > http://lists.linuxtogo.org/cgi-bin/mailman/listinfo/openembedded-core _______________________________________________ Openembedded-core mailing list [email protected] http://lists.linuxtogo.org/cgi-bin/mailman/listinfo/openembedded-core
