From: Martin Jansa <martin.jansa@gmail.com>
To: Richard Purdie <richard.purdie@linuxfoundation.org>
Cc: openembedded-core@lists.openembedded.org
Subject: Re: [PATCH][morty] gcc6: Upgrade to 6.4.0 (latest stable series release)
Date: Sun, 4 Mar 2018 15:15:38 +0100 [thread overview]
Message-ID: <20180304141538.GB1370@jama> (raw)
In-Reply-To: <1520162492-29839-1-git-send-email-richard.purdie@linuxfoundation.org>
[-- Attachment #1: Type: text/plain, Size: 1878 bytes --]
On Sun, Mar 04, 2018 at 11:21:32AM +0000, Richard Purdie wrote:
> From: Juro Bystricky <juro.bystricky@intel.com>
>
> Changed BINV and PV from 6.2.0 to 6.4.0
> Modified BASEURI to load xz tarball instead of bz2
>
> Backported series of patches from https://github.com/hjl-tools/gcc.git
> branch /hjl/indirect/gcc-6-branch/master which contains
> an IA patch series for security related issues
>
> ubsan-fix-check-empty-string.patch : removed, already applied
Very limited testing with this change in morty shows that raspberrypi3
(cortexa7hf-neon-vfpv4) build of glibc-initial fails with:
configure:5344: checking for broken __attribute__((alias()))
configure:5359: arm-oe-linux-gnueabi-gcc -march=armv7ve -marm
-mfpu=neon-vfpv4 -mfloat-abi=hard -mcpu=cortex-a7
--sysroot=sysroots/raspberrypi3-tcbootstrap
-Werror -S conftest.c -o conftest.s 1>&5
conftest.c:1:0: error: switch -mcpu=cortex-a7 conflicts with
-march=armv7ve switch [-Werror]
extern int foo (int x) __asm ("xyzzy");
cc1: all warnings being treated as errors
configure:5362: $? = 1
configure:5372: result: yes
configure:5375: error: working alias attribute support required
Which is a bit strange as armv7ve was added to resolve this error with
-mcpu=cortexa7 and -march=armv7a in:
commit 35392025f3236f5e5393f9cf0857732da9a2e503
Author: Martin Jansa <Martin.Jansa@gmail.com>
Date: Tue Oct 6 17:08:59 2015 +0200
arch-armv7ve: add tune include for armv7ve and use it from cortexa7
and cortexa15
* be aware that this -march value is available only in gcc-4.9 and
newer:
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=57907
* -mcpu=cortex15 and -mcpu=cortexa7 conflict with -march=armv7a
We either have to stop putting -march in default CCARGS or at
least set it compatible one like this patch does.
Regards,
[-- Attachment #2: Digital signature --]
[-- Type: application/pgp-signature, Size: 201 bytes --]
next prev parent reply other threads:[~2018-03-04 14:15 UTC|newest]
Thread overview: 9+ messages / expand[flat|nested] mbox.gz Atom feed top
2018-03-04 11:21 [PATCH][morty] gcc6: Upgrade to 6.4.0 (latest stable series release) Richard Purdie
2018-03-04 11:35 ` ✗ patchtest: failure for " Patchwork
2018-03-04 14:15 ` Martin Jansa [this message]
2018-03-04 16:22 ` [PATCH][morty] " Martin Jansa
2018-03-05 6:37 ` Andre McCurdy
2018-03-05 9:19 ` Martin Jansa
2018-03-05 9:51 ` Andre McCurdy
2018-03-05 10:45 ` Martin Jansa
2018-03-06 2:31 ` Andre McCurdy
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=20180304141538.GB1370@jama \
--to=martin.jansa@gmail.com \
--cc=openembedded-core@lists.openembedded.org \
--cc=richard.purdie@linuxfoundation.org \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox