From 582cfe03cb7496371bc6d83af49b6324d0c36887 Mon Sep 17 00:00:00 2001 From: Ramana Radhakrishnan Date: Thu, 12 Nov 2015 10:50:22 +0000 Subject: [PATCH] Fix PR gas/19217 2015-11-11 Matthew Wahab PR gas/19217 * config/tc-arm.c (move_or_literal_pool): Remove redundant feature check. Fix some code formatting. Drop use of MOVT. Add some comments. 2015-11-11 Matthew Wahab PR gas/19217 * gas/arm/thumb2_ldr_immediate_armv6t2.d: Update expected output. --- gas/ChangeLog | 7 +++++ gas/config/tc-arm.c | 31 +++++++------------ gas/testsuite/ChangeLog | 5 +++ .../gas/arm/thumb2_ldr_immediate_armv6t2.d | 6 ++-- 4 files changed, 26 insertions(+), 23 deletions(-) diff --git a/gas/ChangeLog b/gas/ChangeLog index 10573d5afd..7c00c38dd1 100644 --- a/gas/ChangeLog +++ b/gas/ChangeLog @@ -1,3 +1,10 @@ +2015-11-11 Matthew Wahab + + PR gas/19217 + * config/tc-arm.c (move_or_literal_pool): Remove redundant feature + check. Fix some code formatting. Drop use of MOVT. Add some + comments. + 2015-11-11 Alan Modra Peter Bergner diff --git a/gas/config/tc-arm.c b/gas/config/tc-arm.c index e33a77fb9b..5a8ac95fce 100644 --- a/gas/config/tc-arm.c +++ b/gas/config/tc-arm.c @@ -7847,10 +7847,10 @@ move_or_literal_pool (int i, enum lit_type t, bfd_boolean mode_3) return TRUE; } - if (ARM_CPU_HAS_FEATURE (cpu_variant, arm_arch_t2) - && ARM_CPU_HAS_FEATURE (cpu_variant, arm_ext_v6t2)) + if (ARM_CPU_HAS_FEATURE (cpu_variant, arm_arch_t2)) { - /* Check if on thumb2 it can be done with a mov.w or mvn.w instruction. */ + /* Check if on thumb2 it can be done with a mov.w or mvn.w + instruction. */ unsigned int newimm; bfd_boolean isNegated; @@ -7859,36 +7859,27 @@ move_or_literal_pool (int i, enum lit_type t, bfd_boolean mode_3) isNegated = FALSE; else { - newimm = encode_thumb32_immediate (~ v); + newimm = encode_thumb32_immediate (~v); if (newimm != (unsigned int) FAIL) isNegated = TRUE; } if (newimm != (unsigned int) FAIL) { - inst.instruction = 0xf04f0000 | (inst.operands[i].reg << 8); - inst.instruction |= (isNegated?0x200000:0); + inst.instruction = (0xf04f0000 + | (inst.operands[i].reg << 8)); + inst.instruction |= (isNegated ? 0x200000 : 0); inst.instruction |= (newimm & 0x800) << 15; inst.instruction |= (newimm & 0x700) << 4; inst.instruction |= (newimm & 0x0ff); return TRUE; } - else if ((v & ~0xFFFF) == 0 || (v & ~0xFFFF0000) == 0) + else if ((v & ~0xFFFF) == 0) { - /* The number may be loaded with a movw/movt instruction. */ - int imm; - - if ((inst.reloc.exp.X_add_number & ~0xFFFF) == 0) - { - inst.instruction= 0xf2400000; - imm = v; - } - else - { - inst.instruction = 0xf2c00000; - imm = v >> 16; - } + /* The number can be loaded with a mov.w instruction. */ + int imm = v & 0xFFFF; + inst.instruction = 0xf2400000; /* MOVW. */ inst.instruction |= (inst.operands[i].reg << 8); inst.instruction |= (imm & 0xf000) << 4; inst.instruction |= (imm & 0x0800) << 15; diff --git a/gas/testsuite/ChangeLog b/gas/testsuite/ChangeLog index 119b42ad48..d15d5f5d96 100644 --- a/gas/testsuite/ChangeLog +++ b/gas/testsuite/ChangeLog @@ -1,3 +1,8 @@ +2015-11-11 Matthew Wahab + + PR gas/19217 + * gas/arm/thumb2_ldr_immediate_armv6t2.d: Update expected output. + 2015-11-11 Alan Modra Peter Bergner diff --git a/gas/testsuite/gas/arm/thumb2_ldr_immediate_armv6t2.d b/gas/testsuite/gas/arm/thumb2_ldr_immediate_armv6t2.d index 698371a734..223fbaf6a6 100644 --- a/gas/testsuite/gas/arm/thumb2_ldr_immediate_armv6t2.d +++ b/gas/testsuite/gas/arm/thumb2_ldr_immediate_armv6t2.d @@ -9,7 +9,7 @@ Disassembly of section .text: 0[0-9a-f]+ <[^>]+> f04f 2163 mov.w r1, #1660969728 .* 0[0-9a-f]+ <[^>]+> f04f 1151 mov.w r1, #5308497 .* 0[0-9a-f]+ <[^>]+> f44f 228e mov.w r2, #290816 .* -0[0-9a-f]+ <[^>]+> f6cf 7232 movt r2, #65330 .* +0[0-9a-f]+ <[^>]+> 4a01 ldr r2, \[pc, #4\] .* 0[0-9a-f]+ <[^>]+> f241 32f1 movw r2, #5105 .* - - +0[0-9a-f]+ <[^>]+> 0000 .short 0x0000 +0[0-9a-f]+ <[^>]+> ff320000 .word 0xff320000