[Patch ARM] Fix build attributes for armv8-a in case of assembler files that contain...
authorRamana Radhakrishnan <ramana.radhakrishnan@arm.com>
Thu, 17 Dec 2015 10:55:54 +0000 (10:55 +0000)
committerRamana Radhakrishnan <ramana.radhakrishnan@arm.com>
Thu, 17 Dec 2015 11:34:39 +0000 (11:34 +0000)
There is currently a problem in the way in which we produce
build attributes for simple assembler files that have armv8-a
instructions.

In these case we need to generate TAG_ISA_THUMB_Use to be Thumb-2
and set the architecture profile to be 'A' rather than not
setting architecture profile to be 'A' and setting TAG_ISA_THUMB_Use
to be Thumb-1.

This is a pre-requisite for any v8-m patches that have been posted.
arm-none-eabi gas testsuite run. no regressions.

2015-12-17  Ramana Radhakrishnan  <ramana.radhakrishnan@arm.com>

* gas/config/tc-arm.c (aeabi_set_public_attributes): Adjust
TAG_ARCH_profile for armv8-a.
* gas/testsuite/gas/arm/armv8a-automatic-hlt.d: New test.
* gas/testsuite/gas/arm/armv8a-automatic-hlt.s: New test.
* gas/testsuite/gas/arm/armv8a-automatic-lda.d: New test.
* gas/testsuite/gas/arm/armv8a-automatic-lda.s: New test.

gas/config/tc-arm.c
gas/testsuite/gas/arm/armv8a-automatic-hlt.d [new file with mode: 0644]
gas/testsuite/gas/arm/armv8a-automatic-hlt.s [new file with mode: 0644]
gas/testsuite/gas/arm/armv8a-automatic-lda.d [new file with mode: 0644]
gas/testsuite/gas/arm/armv8a-automatic-lda.s [new file with mode: 0644]
include/opcode/arm.h

index 6f3f93328cf0a4d37fbd8851fb7ea1d1e3a62a0c..f9c76ef7c35960cc06d89d842d79024a6a0d7652 100644 (file)
@@ -25649,7 +25649,8 @@ aeabi_set_public_attributes (void)
   aeabi_set_attribute_int (Tag_CPU_arch, arch);
 
   /* Tag_CPU_arch_profile.  */
-  if (ARM_CPU_HAS_FEATURE (flags, arm_ext_v7a))
+  if (ARM_CPU_HAS_FEATURE (flags, arm_ext_v7a)
+      || ARM_CPU_HAS_FEATURE (flags, arm_ext_v8))
     profile = 'A';
   else if (ARM_CPU_HAS_FEATURE (flags, arm_ext_v7r))
     profile = 'R';
diff --git a/gas/testsuite/gas/arm/armv8a-automatic-hlt.d b/gas/testsuite/gas/arm/armv8a-automatic-hlt.d
new file mode 100644 (file)
index 0000000..6e8c1db
--- /dev/null
@@ -0,0 +1,8 @@
+# readelf: -A
+# This test is only valid on ELF based ports.
+#not-target: *-*-*coff *-*-pe *-*-wince *-*-*aout* *-*-netbsd *-*-riscix*
+Attribute Section: aeabi
+File Attributes
+  Tag_CPU_arch: v8
+  Tag_CPU_arch_profile: Application
+  Tag_THUMB_ISA_use: Thumb-2
diff --git a/gas/testsuite/gas/arm/armv8a-automatic-hlt.s b/gas/testsuite/gas/arm/armv8a-automatic-hlt.s
new file mode 100644 (file)
index 0000000..29dd4ff
--- /dev/null
@@ -0,0 +1,6 @@
+       .thumb
+       .syntax unified
+       hlt
+       stlexd r0, r2, [r3]
+       sevl
+       hlt 0xf
diff --git a/gas/testsuite/gas/arm/armv8a-automatic-lda.d b/gas/testsuite/gas/arm/armv8a-automatic-lda.d
new file mode 100644 (file)
index 0000000..6e8c1db
--- /dev/null
@@ -0,0 +1,8 @@
+# readelf: -A
+# This test is only valid on ELF based ports.
+#not-target: *-*-*coff *-*-pe *-*-wince *-*-*aout* *-*-netbsd *-*-riscix*
+Attribute Section: aeabi
+File Attributes
+  Tag_CPU_arch: v8
+  Tag_CPU_arch_profile: Application
+  Tag_THUMB_ISA_use: Thumb-2
diff --git a/gas/testsuite/gas/arm/armv8a-automatic-lda.s b/gas/testsuite/gas/arm/armv8a-automatic-lda.s
new file mode 100644 (file)
index 0000000..8e1ca0c
--- /dev/null
@@ -0,0 +1,4 @@
+       .thumb
+       .syntax unified
+       lda r0, [r0]
+       stl r0, [r1]
index 0814d806a5b2cf90fc86d36a499950364c063ab5..daeb626ee2282c127c2ab5d24a8315fe55c75cae 100644 (file)
 #define ARM_ANY                ARM_FEATURE (-1, -1, 0) /* Any basic core.  */
 #define ARM_FEATURE_ALL        ARM_FEATURE (-1, -1, -1)/* All CPU and FPU features.  */
 #define FPU_ANY_HARD   ARM_FEATURE_COPROC (FPU_FPA | FPU_VFP_HARD | FPU_MAVERICK)
-#define ARM_ARCH_THUMB2 ARM_FEATURE_CORE_LOW (ARM_EXT_V6T2 | ARM_EXT_V7 \
+#define ARM_ARCH_THUMB2 ARM_FEATURE_CORE_LOW (ARM_EXT_V6T2 | ARM_EXT_V7        \
                                              | ARM_EXT_V7A | ARM_EXT_V7R \
-                                             | ARM_EXT_V7M | ARM_EXT_DIV)
+                                             | ARM_EXT_V7M | ARM_EXT_DIV \
+                                             | ARM_EXT_V8)
 /* v7-a+sec.  */
 #define ARM_ARCH_V7A_SEC ARM_FEATURE_CORE_LOW (ARM_AEXT_V7A | ARM_EXT_SEC)
 /* v7-a+mp+sec.  */