[OE-core] [PATCH 1/2] armv8: update to use armv8-a tune

Denys Dmytriyenko denis at denix.org
Thu Aug 3 20:25:59 UTC 2017


On Mon, Jul 31, 2017 at 12:31:58PM -0500, Mark Hatle wrote:
> On 7/31/17 12:16 PM, Armin Kuster wrote:
> > Signed-off-by: Armin Kuster <akuster808 at gmail.com>
> > ---
> >  meta/conf/machine/include/arm/arch-armv8.inc | 25 +++++++++++++++++++++++++
> >  1 file changed, 25 insertions(+)
> > 
> > diff --git a/meta/conf/machine/include/arm/arch-armv8.inc b/meta/conf/machine/include/arm/arch-armv8.inc
> > index 5e832fa..dc1ba5e 100644
> > --- a/meta/conf/machine/include/arm/arch-armv8.inc
> > +++ b/meta/conf/machine/include/arm/arch-armv8.inc
> > @@ -1 +1,26 @@
> > +DEFAULTTUNE ?= "armv8-a"
> 
> do we want the '-a'?  The other arm (7) are of the format armv7a (no '-').

On the other hand, gcc went with armv7-a instead of armv7a...
So, maybe having '-a' is actually more aligned with others?


> > +
> > +TUNEVALID[armv8-a] = "Enable instructions for ARMv8-a"
> > +TUNE_CCARGS .= "${@bb.utils.contains('TUNE_FEATURES', 'armv8-a', ' -march=armv8-a', '', d)}"
> > +
> > +TUNEVALID[crypto] = "Enable crypto instructions for ARMv8-a"
> > +TUNE_CCARGS .= "${@bb.utils.contains('TUNE_FEATURES', 'crypto', '+crypto', '', d)}"
> > +
> > +MACHINEOVERRIDES =. "${@bb.utils.contains('TUNE_FEATURES', 'armv8-a', 'armv8a:', '' ,d)}"
> > +
> >  require conf/machine/include/arm/arch-arm64.inc
> > +
> > +AVAILTUNES += "armv8-a armv8-a-crypto"
> > +ARMPKGARCH_tune-armv8-a ?= "armv8-a"
> > +
> > +# Enable Crypto extension. This also enables Advanced SIMD and floating-point instructions
> > +ARMPKGARCH_tune-armv8-a-crypto ?= "armv8-a"
> > +
> > +TUNE_FEATURES_tune-armv8-a = "aarch64 armv8-a"
> > +TUNE_FEATURES_tune-armv8-a-crypto = "${TUNE_FEATURES_tune-armv8-a} crypto"
> > +
> > +PACKAGE_EXTRA_ARCHS_tune-armv8-a = "aarch64 armv8-a"
> > +PACKAGE_EXTRA_ARCHS_tune-armv8-a-crypto = "${PACKAGE_EXTRA_ARCHS_tune-armv8-a} armv8-a-crypto"
> > +
> > +BASE_LIB_tune-armv8-a = "${BASE_LIB_tune-aarch64}"
> > +BASE_LIB_tune-armv8-a-crypto = "${BASE_LIB_tune-aarch64}"
> > 
> 
> -- 
> _______________________________________________
> Openembedded-core mailing list
> Openembedded-core at lists.openembedded.org
> http://lists.openembedded.org/mailman/listinfo/openembedded-core



More information about the Openembedded-core mailing list