<div><br></div><div><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">On Tue, Mar 19, 2019 at 4:45 AM Adrian Bunk <<a href="mailto:bunk@stusta.de">bunk@stusta.de</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">SIMD instructions are a mandatory part of armv8a<br>
(they were optional in armv7a), and the gcc docs<br>
also say that they are always enabled.</blockquote><div dir="auto"><br></div><div dir="auto">This should be ok but please cite the reference to arch specs where it’s mentioned and if someone</div><div dir="auto">Has need to turn it off then they should chime back </div><div dir="auto">Since removing this just removes a detail which we can actually keep without much effort</div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><br>
<br>
Signed-off-by: Adrian Bunk <<a href="mailto:bunk@stusta.de" target="_blank">bunk@stusta.de</a>><br>
---<br>
meta/conf/machine/include/arm/arch-armv8a.inc | 6 ++----<br>
meta/conf/machine/include/tune-cortexa32.inc | 4 ++--<br>
meta/conf/machine/include/tune-cortexa35.inc | 4 ++--<br>
meta/conf/machine/include/tune-cortexa53.inc | 4 ++--<br>
meta/conf/machine/include/tune-cortexa72.inc | 2 +-<br>
5 files changed, 9 insertions(+), 11 deletions(-)<br>
<br>
diff --git a/meta/conf/machine/include/arm/arch-armv8a.inc b/meta/conf/machine/include/arm/arch-armv8a.inc<br>
index 22064559be..44d0ca4557 100644<br>
--- a/meta/conf/machine/include/arm/arch-armv8a.inc<br>
+++ b/meta/conf/machine/include/arm/arch-armv8a.inc<br>
@@ -2,8 +2,6 @@ DEFAULTTUNE ?= "armv8a-crc"<br>
<br>
TUNEVALID[armv8a] = "Enable instructions for ARMv8-a"<br>
TUNE_CCARGS .= "${@bb.utils.contains('TUNE_FEATURES', 'armv8a', ' -march=armv8-a', '', d)}"<br>
-TUNEVALID[simd] = "Enable instructions for ARMv8-a Advanced SIMD and floating-point"<br>
-TUNE_CCARGS .= "${@bb.utils.contains('TUNE_FEATURES', 'simd', '+simd', '', d)}"<br>
TUNEVALID[crc] = "Enable instructions for ARMv8-a Cyclic Redundancy Check (CRC)"<br>
TUNE_CCARGS .= "${@bb.utils.contains('TUNE_FEATURES', 'crc', '+crc', '', d)}"<br>
TUNEVALID[crypto] = "Enable instructions for ARMv8-a cryptographic"<br>
@@ -18,11 +16,11 @@ ARMPKGARCH_tune-armv8a ?= "armv8a"<br>
ARMPKGARCH_tune-armv8a-crc ?= "armv8a"<br>
ARMPKGARCH_tune-armv8a-crypto ?= "armv8a"<br>
ARMPKGARCH_tune-armv8a-crc-crypto ?= "armv8a"<br>
-TUNE_FEATURES_tune-armv8a = "aarch64 armv8a simd"<br>
+TUNE_FEATURES_tune-armv8a = "aarch64 armv8a"<br>
TUNE_FEATURES_tune-armv8a-crc = "${TUNE_FEATURES_tune-armv8a} crc"<br>
TUNE_FEATURES_tune-armv8a-crypto = "${TUNE_FEATURES_tune-armv8a} crypto"<br>
TUNE_FEATURES_tune-armv8a-crc-crypto = "${TUNE_FEATURES_tune-armv8a-crc} crypto"<br>
-PACKAGE_EXTRA_ARCHS_tune-armv8a = "aarch64 armv8a simd"<br>
+PACKAGE_EXTRA_ARCHS_tune-armv8a = "aarch64 armv8a"<br>
PACKAGE_EXTRA_ARCHS_tune-armv8a-crc = "${PACKAGE_EXTRA_ARCHS_tune-armv8a} crc"<br>
PACKAGE_EXTRA_ARCHS_tune-armv8a-crypto = "${PACKAGE_EXTRA_ARCHS_tune-armv8a} crypto"<br>
PACKAGE_EXTRA_ARCHS_tune-armv8a-crc-crypto = "${PACKAGE_EXTRA_ARCHS_tune-armv8a-crc} crypto"<br>
diff --git a/meta/conf/machine/include/tune-cortexa32.inc b/meta/conf/machine/include/tune-cortexa32.inc<br>
index 4bf7f774e1..9c948f1766 100644<br>
--- a/meta/conf/machine/include/tune-cortexa32.inc<br>
+++ b/meta/conf/machine/include/tune-cortexa32.inc<br>
@@ -10,8 +10,8 @@ require conf/machine/include/arm/arch-armv8a.inc<br>
AVAILTUNES += "cortexa32 cortexa32-crypto"<br>
ARMPKGARCH_tune-cortexa32 = "cortexa32"<br>
ARMPKGARCH_tune-cortexa32-crypto = "cortexa32"<br>
-TUNE_FEATURES_tune-cortexa32 = "aarch64 cortexa32 simd crc"<br>
-TUNE_FEATURES_tune-cortexa32-crypto = "aarch64 cortexa32 simd crc crypto"<br>
+TUNE_FEATURES_tune-cortexa32 = "aarch64 cortexa32 crc"<br>
+TUNE_FEATURES_tune-cortexa32-crypto = "aarch64 cortexa32 crc crypto"<br>
PACKAGE_EXTRA_ARCHS_tune-cortexa32 = "${PACKAGE_EXTRA_ARCHS_tune-armv8a-crc} cortexa32"<br>
PACKAGE_EXTRA_ARCHS_tune-cortexa32-crypto = "${PACKAGE_EXTRA_ARCHS_tune-armv8a-crc-crypto} cortexa32 cortexa32-crypto"<br>
BASE_LIB_tune-cortexa32 = "lib64"<br>
diff --git a/meta/conf/machine/include/tune-cortexa35.inc b/meta/conf/machine/include/tune-cortexa35.inc<br>
index 6ece810753..d6df6cd6bd 100644<br>
--- a/meta/conf/machine/include/tune-cortexa35.inc<br>
+++ b/meta/conf/machine/include/tune-cortexa35.inc<br>
@@ -9,8 +9,8 @@ require conf/machine/include/arm/arch-armv8a.inc<br>
AVAILTUNES += "cortexa35 cortexa35-crypto"<br>
ARMPKGARCH_tune-cortexa35 = "cortexa35"<br>
ARMPKGARCH_tune-cortexa35-crypto = "cortexa35"<br>
-TUNE_FEATURES_tune-cortexa35 = "aarch64 cortexa35 simd crc"<br>
-TUNE_FEATURES_tune-cortexa35-crypto = "aarch64 cortexa35 simd crc crypto"<br>
+TUNE_FEATURES_tune-cortexa35 = "aarch64 cortexa35 crc"<br>
+TUNE_FEATURES_tune-cortexa35-crypto = "aarch64 cortexa35 crc crypto"<br>
PACKAGE_EXTRA_ARCHS_tune-cortexa35 = "${PACKAGE_EXTRA_ARCHS_tune-armv8a-crc} cortexa35"<br>
PACKAGE_EXTRA_ARCHS_tune-cortexa35-crypto = "${PACKAGE_EXTRA_ARCHS_tune-armv8a-crc-crypto} cortexa35 cortexa35-crypto"<br>
BASE_LIB_tune-cortexa35 = "lib64"<br>
diff --git a/meta/conf/machine/include/tune-cortexa53.inc b/meta/conf/machine/include/tune-cortexa53.inc<br>
index 9fb37ff639..6c0b443eaf 100644<br>
--- a/meta/conf/machine/include/tune-cortexa53.inc<br>
+++ b/meta/conf/machine/include/tune-cortexa53.inc<br>
@@ -9,8 +9,8 @@ require conf/machine/include/arm/arch-armv8a.inc<br>
AVAILTUNES += "cortexa53 cortexa53-crypto"<br>
ARMPKGARCH_tune-cortexa53 = "cortexa53"<br>
ARMPKGARCH_tune-cortexa53-crypto = "cortexa53"<br>
-TUNE_FEATURES_tune-cortexa53 = "aarch64 cortexa53 simd crc"<br>
-TUNE_FEATURES_tune-cortexa53-crypto = "aarch64 cortexa53 simd crc crypto"<br>
+TUNE_FEATURES_tune-cortexa53 = "aarch64 cortexa53 crc"<br>
+TUNE_FEATURES_tune-cortexa53-crypto = "aarch64 cortexa53 crc crypto"<br>
PACKAGE_EXTRA_ARCHS_tune-cortexa53 = "${PACKAGE_EXTRA_ARCHS_tune-armv8a-crc} cortexa53"<br>
PACKAGE_EXTRA_ARCHS_tune-cortexa53-crypto = "${PACKAGE_EXTRA_ARCHS_tune-armv8a-crc-crypto} cortexa53 cortexa53-crypto"<br>
BASE_LIB_tune-cortexa53 = "lib64"<br>
diff --git a/meta/conf/machine/include/tune-cortexa72.inc b/meta/conf/machine/include/tune-cortexa72.inc<br>
index 4c2306c4fd..00f7745a22 100644<br>
--- a/meta/conf/machine/include/tune-cortexa72.inc<br>
+++ b/meta/conf/machine/include/tune-cortexa72.inc<br>
@@ -8,6 +8,6 @@ require conf/machine/include/arm/arch-armv8a.inc<br>
# Little Endian base configs<br>
AVAILTUNES += "cortexa72"<br>
ARMPKGARCH_tune-cortexa72 = "cortexa72"<br>
-TUNE_FEATURES_tune-cortexa72 = "aarch64 cortexa72 simd crc crypto"<br>
+TUNE_FEATURES_tune-cortexa72 = "aarch64 cortexa72 crc crypto"<br>
PACKAGE_EXTRA_ARCHS_tune-cortexa72 = "${PACKAGE_EXTRA_ARCHS_tune-armv8a-crc-crypto} cortexa72"<br>
BASE_LIB_tune-cortexa72 = "lib64"<br>
-- <br>
2.17.1<br>
<br>
-- <br>
_______________________________________________<br>
Openembedded-core mailing list<br>
<a href="mailto:Openembedded-core@lists.openembedded.org" target="_blank">Openembedded-core@lists.openembedded.org</a><br>
<a href="http://lists.openembedded.org/mailman/listinfo/openembedded-core" rel="noreferrer" target="_blank">http://lists.openembedded.org/mailman/listinfo/openembedded-core</a><br>
</blockquote></div></div>