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

[Mark Hatle]

On 7/31/17 12:40 PM, akuster808 wrote:
> 
> 
> On 07/31/2017 10:31 AM, 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 '-').
> works for me either way.
> 
> While we are at it. How would we want ‘armv8.1-a’, ‘armv8.2-a’, ‘armv8.3-a'
> formated as?

My preference is to drop the '-'.  As for the '.', I'm not sure.. not something
we've run across before.

We could just drop it (the '.'), but it really depends on if armv81a would
confuse someone (familiar with arm) or not.

--Mark

> - armin
>>
>> --Mark
>>
>>> +
>>> +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}"
>>>
>