Re: [PATCHv2 09/16] cortexa{7,15,17}: add VFPv4 tunes

[Martin Jansa <martin.jansa@gmail.com>]

[-- Attachment #1: Type: text/plain, Size: 1404 bytes --]

On Wed, Nov 18, 2015 at 04:36:04PM +0000, Phil Blundell wrote:
> On Tue, 2015-11-17 at 11:58 +0100, Martin Jansa wrote:
> > * it was added only to hf cortexa7 in:
> >   commit e97d152ca13556b41a236c1a4cfb11e77ff857d7
> >   Author: Kristof Robot <krirobo@gmail.com>
> >   Date:   Sun Jan 26 10:03:56 2014 +0100
> > 
> >       Add Cortex A7 support for NEONv2 & FPv4
> > 
> > * add it to softfp cortexa7 and both versions for cortexa15 and
> >   cortexa17 tunes
> 
> I can't say I'm wildly thrilled about the combinatorial explosion of
> tunes that differ only in a few letters.  Can't we just enable NEON and
> VFPv4 under the existing cortex-a15 tune?  Those ISAs are both part of
> the Cortex-A15 cpu, right?

I don't have strong opinion about this, I just want it to behave
consistently.

> Also...
> 
> > * we should also update Cortex-A7 and Cortex-A15, to use 
> > -march=armv7ve,
> >   but the problem is that oe-core has gcc-4.[89] and gcc-5.2 and gcc
> > -4.8
> >   doesn't support it yet, maybe we should do this change after gcc
> > -4.8
> >   is removed, for details see:
> >   https://gcc.gnu.org/bugzilla/show_bug.cgi?id=57907
> 
> ... this part of the commit message is superseded by the following
> patch, right?

Correct, I'll remove it and resend when there is some decision about
above.

-- 
Martin 'JaMa' Jansa     jabber: Martin.Jansa@gmail.com

[-- Attachment #2: Digital signature --]
[-- Type: application/pgp-signature, Size: 188 bytes --]