From afa62d5e344305d36b8af043030fd6d69af850db Mon Sep 17 00:00:00 2001 From: Daniel Jacobowitz Date: Wed, 13 Jan 2010 19:01:10 +0000 Subject: [PATCH] gas/ * config/tc-arm.c (do_t_nop): Correct check for Thumb-2 NOP. gas/testsuite/ * gas/arm/thumb-nop.d, gas/arm/thumb-nop.s: New test. * gas/arm/relax_branch_align.d: Expect a default NOP instruction. * gas/arm/vfp1_t2.d, gas/arm/vfp1xD_t2.d: Specify a CPU with Thumb-2. ld/testsuite/ * ld-arm/arm-elf.exp (armelftests): Assemble Cortex-A8 tests with -mcpu=cortex-a8. --- gas/ChangeLog | 4 ++++ gas/config/tc-arm.c | 2 +- gas/testsuite/ChangeLog | 7 +++++++ gas/testsuite/gas/arm/relax_branch_align.d | 8 ++++---- gas/testsuite/gas/arm/thumb-nop.d | 11 +++++++++++ gas/testsuite/gas/arm/thumb-nop.s | 5 +++++ gas/testsuite/gas/arm/vfp1_t2.d | 2 +- gas/testsuite/gas/arm/vfp1xD_t2.d | 2 +- ld/testsuite/ChangeLog | 5 +++++ ld/testsuite/ld-arm/arm-elf.exp | 6 ++++-- 10 files changed, 43 insertions(+), 9 deletions(-) create mode 100644 gas/testsuite/gas/arm/thumb-nop.d create mode 100644 gas/testsuite/gas/arm/thumb-nop.s diff --git a/gas/ChangeLog b/gas/ChangeLog index f899ae98e5..8ae07df428 100644 --- a/gas/ChangeLog +++ b/gas/ChangeLog @@ -1,3 +1,7 @@ +2010-01-13 Daniel Jacobowitz + + * config/tc-arm.c (do_t_nop): Correct check for Thumb-2 NOP. + 2010-01-13 Nick Clifton * config/tc-h8300.c (h8300_elf_section): New function - issue a diff --git a/gas/config/tc-arm.c b/gas/config/tc-arm.c index b0184a5c80..d5a0ec4b12 100644 --- a/gas/config/tc-arm.c +++ b/gas/config/tc-arm.c @@ -10623,7 +10623,7 @@ do_t_nop (void) { /* PR9722: Check for Thumb2 availability before generating a thumb2 nop instruction. */ - if (ARM_CPU_HAS_FEATURE (cpu_variant, arm_arch_t2)) + if (ARM_CPU_HAS_FEATURE (selected_cpu, arm_ext_v6t2)) { inst.instruction = THUMB_OP16 (inst.instruction); inst.instruction |= inst.operands[0].imm << 4; diff --git a/gas/testsuite/ChangeLog b/gas/testsuite/ChangeLog index 2fafd115df..64d3cc7774 100644 --- a/gas/testsuite/ChangeLog +++ b/gas/testsuite/ChangeLog @@ -1,3 +1,10 @@ +2010-01-13 Daniel Jacobowitz + + * gas/arm/thumb-nop.d, gas/arm/thumb-nop.s: New test. + * gas/arm/relax_branch_align.d: Expect a default NOP instruction. + * gas/arm/vfp1_t2.d, gas/arm/vfp1xD_t2.d: Specify a CPU with + Thumb-2. + 2010-01-13 Nick Clifton * gas/elf/section0.d: Skip this test for the h8300. diff --git a/gas/testsuite/gas/arm/relax_branch_align.d b/gas/testsuite/gas/arm/relax_branch_align.d index e23b095158..e19857a587 100644 --- a/gas/testsuite/gas/arm/relax_branch_align.d +++ b/gas/testsuite/gas/arm/relax_branch_align.d @@ -4,10 +4,10 @@ .*: +file format .*arm.* Disassembly of section .text: -0+000 <[^>]+> bf00 nop +0+000 <[^>]+> 46c0 nop ; \(mov r8, r8\) 0+002 <[^>]+> f000 8080 beq.w 0+106 <[^>]*> -0+006 <[^>]+> bf00 nop +0+006 <[^>]+> 46c0 nop ; \(mov r8, r8\) #... -0+100 <[^>]+> bf00 nop +0+100 <[^>]+> 46c0 nop ; \(mov r8, r8\) 0+102 <[^>]+> f47f af80 bne.w 0+006 <[^>]*> -0+106 <[^>]+> bf00 nop +0+106 <[^>]+> 46c0 nop ; \(mov r8, r8\) diff --git a/gas/testsuite/gas/arm/thumb-nop.d b/gas/testsuite/gas/arm/thumb-nop.d new file mode 100644 index 0000000000..648ed986a3 --- /dev/null +++ b/gas/testsuite/gas/arm/thumb-nop.d @@ -0,0 +1,11 @@ +# name: Thumb NOP +# objdump: -dr --prefix-addresses --show-raw-insn +# +# Both explicit nop and padding should not use Thumb-2 NOP for the +# default CPU. + +.*: +file format .*arm.* + +Disassembly of section \.text: +0+000 <[^>]+> 46c0 nop ; \(mov r8, r8\) +0+002 <[^>]+> 46c0 nop ; \(mov r8, r8\) diff --git a/gas/testsuite/gas/arm/thumb-nop.s b/gas/testsuite/gas/arm/thumb-nop.s new file mode 100644 index 0000000000..55b7b6c645 --- /dev/null +++ b/gas/testsuite/gas/arm/thumb-nop.s @@ -0,0 +1,5 @@ + .text + .code 16 + .p2align 2 +.foo: + nop diff --git a/gas/testsuite/gas/arm/vfp1_t2.d b/gas/testsuite/gas/arm/vfp1_t2.d index 2ff52fdecd..757bf10323 100644 --- a/gas/testsuite/gas/arm/vfp1_t2.d +++ b/gas/testsuite/gas/arm/vfp1_t2.d @@ -1,6 +1,6 @@ #objdump: -dr --prefix-addresses --show-raw-insn #name: Thumb-2 VFP Double-precision instructions -#as: -mfpu=vfp +#as: -mfpu=vfp -mcpu=arm1156t2f-s # Test the ARM VFP Double Precision instructions diff --git a/gas/testsuite/gas/arm/vfp1xD_t2.d b/gas/testsuite/gas/arm/vfp1xD_t2.d index 4cbadec453..7dd5030f28 100644 --- a/gas/testsuite/gas/arm/vfp1xD_t2.d +++ b/gas/testsuite/gas/arm/vfp1xD_t2.d @@ -1,6 +1,6 @@ #objdump: -dr --prefix-addresses --show-raw-insn #name: Thumb-2 VFP Single-precision instructions -#as: -mfpu=vfpxd +#as: -mfpu=vfpxd -mcpu=arm1156t2f-s # Test the ARM VFP Single Precision instructions diff --git a/ld/testsuite/ChangeLog b/ld/testsuite/ChangeLog index dbd6fc143f..7fda06e759 100644 --- a/ld/testsuite/ChangeLog +++ b/ld/testsuite/ChangeLog @@ -1,3 +1,8 @@ +2010-01-13 Daniel Jacobowitz + + * ld-arm/arm-elf.exp (armelftests): Assemble Cortex-A8 tests with + -mcpu=cortex-a8. + 2010-01-13 Nick Clifton * ld-scrips/sort.exp: Skip these tests when the target is the diff --git a/ld/testsuite/ld-arm/arm-elf.exp b/ld/testsuite/ld-arm/arm-elf.exp index ce8e9ee87f..050471d6e0 100644 --- a/ld/testsuite/ld-arm/arm-elf.exp +++ b/ld/testsuite/ld-arm/arm-elf.exp @@ -207,11 +207,13 @@ set armelftests { {{objdump -dr cortex-a8-fix-blx-rel-thumb.d}} "cortex-a8-fix-blx-rel-thumb"} {"Cortex-A8 erratum fix, relocate bl.w and far call" - "-EL -Ttext=0x00 --fix-cortex-a8 --defsym far_fn1=0x80000000 --defsym far_fn2=0x80000004 --defsym far_fn=0x7fff0000 --defsym _start=0" "-EL" {cortex-a8-far-1.s cortex-a8-far-2.s} + "-EL -Ttext=0x00 --fix-cortex-a8 --defsym far_fn1=0x80000000 --defsym far_fn2=0x80000004 --defsym far_fn=0x7fff0000 --defsym _start=0" + "-EL -mcpu=cortex-a8" {cortex-a8-far-1.s cortex-a8-far-2.s} {{objdump -dr cortex-a8-far.d}} "cortex-a8-far"} {"Cortex-A8 erratum fix, headers" - "-EL --fix-cortex-a8 -T cortex-a8-fix-hdr.t" "-EL" {cortex-a8-fix-hdr.s} + "-EL --fix-cortex-a8 -T cortex-a8-fix-hdr.t" + "-EL -mcpu=cortex-a8" {cortex-a8-fix-hdr.s} {{objdump -dr cortex-a8-fix-hdr.d}} "cortex-a8-fix-hdr"} {"Unwinding and -gc-sections" "-gc-sections" "" {gc-unwind.s} -- 2.34.1