* [binutils][objdump][ARM] Fixes for unpredictable nops and 26-bit versions of teq,tst,cmn,cmp
@ 2015-08-05 16:58 Andre Vieira
2015-08-11 16:18 ` Nick Clifton
0 siblings, 1 reply; 5+ messages in thread
From: Andre Vieira @ 2015-08-05 16:58 UTC (permalink / raw)
To: binutils
[-- Attachment #1: Type: text/plain, Size: 978 bytes --]
NOPS with invalid SBO/SBZ fields were being wrongfully translated to
TEQ's. The 26-bit versions of TEQ/TST/CMN/CMP are now marked
UNPREDICTABLE for ARM V6 and higher. See the definition of UNPREDICTABLE
instructions and instruction definitions for TEQ/TST/CMN/CMP in the ARM
Architecture Reference Manual ARMv7-A and ARMv7-R edition, reference ARM
DDI0406C, on infocenter.arm.com.
opcodes/ChangeLog:
2015-07-16 Andre Vieira <andre.simoesdiasvieira@arm.com>
* arm-dis.c (print_insn_arm): Disassembling for all targets V6
and higher with ARM instruction set will now mark the 26-bit
versions of teq,tst,cmn and cmp as UNPREDICTABLE.
(arm_opcodes): Fix for unpredictable nop being recognized as a teq.
gas/testsuite/ChangeLog:
2015-07-21 Andre Vieira <andre.simoesdiasvieira@arm.com>
* gas/arm/nops.d: New.
* gas/arm/nops.s: New.
* gas/arm/inst.d: Changed expectation file for 26-bit teq,
tst, cmn and cmp.
[-- Warning: decoded text below may be mangled, UTF-8 assumed --]
[-- Attachment #2: 0001-fix-for-unpredictable-nops-and-26-bit-teq-cmn-cmp-ts.patch --]
[-- Type: text/x-patch; name=0001-fix-for-unpredictable-nops-and-26-bit-teq-cmn-cmp-ts.patch, Size: 5762 bytes --]
From 2bc83447832981cb9f4f54c95a79c446b0a1133c Mon Sep 17 00:00:00 2001
From: Andre Simoes Dias Vieira <andsim01@arm.com>
Date: Thu, 23 Jul 2015 11:15:24 +0100
Subject: [PATCH] fix for unpredictable nops and 26 bit teq cmn cmp tst
---
gas/testsuite/gas/arm/inst.d | 32 ++++++++++++++++----------------
gas/testsuite/gas/arm/nops.d | 11 +++++++++++
gas/testsuite/gas/arm/nops.s | 4 ++++
opcodes/arm-dis.c | 14 +++++++++++---
4 files changed, 42 insertions(+), 19 deletions(-)
create mode 100644 gas/testsuite/gas/arm/nops.d
create mode 100644 gas/testsuite/gas/arm/nops.s
diff --git a/gas/testsuite/gas/arm/inst.d b/gas/testsuite/gas/arm/inst.d
index e298c5fe41b401dcdb3d4ef87da1bdf58786b480..d7ca4a3ce849980a7524c5875a74ca436727993c 100644
--- a/gas/testsuite/gas/arm/inst.d
+++ b/gas/testsuite/gas/arm/inst.d
@@ -95,22 +95,22 @@ Disassembly of section .text:
0+14c <[^>]*> e1720004 ? cmn r2, r4
0+150 <[^>]*> e1750287 ? cmn r5, r7, lsl #5
0+154 <[^>]*> e1710113 ? cmn r1, r3, lsl r1
-0+158 <[^>]*> e330f00a ? teq r0, #10
-0+15c <[^>]*> e132f004 ? teq r2, r4
-0+160 <[^>]*> e135f287 ? teq r5, r7, lsl #5
-0+164 <[^>]*> e131f113 ? teq r1, r3, lsl r1
-0+168 <[^>]*> e370f00a ? cmn r0, #10
-0+16c <[^>]*> e172f004 ? cmn r2, r4
-0+170 <[^>]*> e175f287 ? cmn r5, r7, lsl #5
-0+174 <[^>]*> e171f113 ? cmn r1, r3, lsl r1
-0+178 <[^>]*> e350f00a ? cmp r0, #10
-0+17c <[^>]*> e152f004 ? cmp r2, r4
-0+180 <[^>]*> e155f287 ? cmp r5, r7, lsl #5
-0+184 <[^>]*> e151f113 ? cmp r1, r3, lsl r1
-0+188 <[^>]*> e310f00a ? tst r0, #10
-0+18c <[^>]*> e112f004 ? tst r2, r4
-0+190 <[^>]*> e115f287 ? tst r5, r7, lsl #5
-0+194 <[^>]*> e111f113 ? tst r1, r3, lsl r1
+0+158 <[^>]*> e330f00a ? teq r0, #10 ; <UNPREDICTABLE>
+0+15c <[^>]*> e132f004 ? teq r2, r4 ; <UNPREDICTABLE>
+0+160 <[^>]*> e135f287 ? teq r5, r7, lsl #5 ; <UNPREDICTABLE>
+0+164 <[^>]*> e131f113 ? teq r1, r3, lsl r1 ; <UNPREDICTABLE>
+0+168 <[^>]*> e370f00a ? cmn r0, #10 ; <UNPREDICTABLE>
+0+16c <[^>]*> e172f004 ? cmn r2, r4 ; <UNPREDICTABLE>
+0+170 <[^>]*> e175f287 ? cmn r5, r7, lsl #5 ; <UNPREDICTABLE>
+0+174 <[^>]*> e171f113 ? cmn r1, r3, lsl r1 ; <UNPREDICTABLE>
+0+178 <[^>]*> e350f00a ? cmp r0, #10 ; <UNPREDICTABLE>
+0+17c <[^>]*> e152f004 ? cmp r2, r4 ; <UNPREDICTABLE>
+0+180 <[^>]*> e155f287 ? cmp r5, r7, lsl #5 ; <UNPREDICTABLE>
+0+184 <[^>]*> e151f113 ? cmp r1, r3, lsl r1 ; <UNPREDICTABLE>
+0+188 <[^>]*> e310f00a ? tst r0, #10 ; <UNPREDICTABLE>
+0+18c <[^>]*> e112f004 ? tst r2, r4 ; <UNPREDICTABLE>
+0+190 <[^>]*> e115f287 ? tst r5, r7, lsl #5 ; <UNPREDICTABLE>
+0+194 <[^>]*> e111f113 ? tst r1, r3, lsl r1 ; <UNPREDICTABLE>
0+198 <[^>]*> e0000291 ? mul r0, r1, r2
0+19c <[^>]*> e0110392 ? muls r1, r2, r3
0+1a0 <[^>]*> 10000091 ? mulne r0, r1, r0
diff --git a/gas/testsuite/gas/arm/nops.d b/gas/testsuite/gas/arm/nops.d
new file mode 100644
index 0000000000000000000000000000000000000000..54ed13ab75ce576f9c08e07bab11c38db68ce88d
--- /dev/null
+++ b/gas/testsuite/gas/arm/nops.d
@@ -0,0 +1,11 @@
+# name: NOP<c> instructions
+# objdump: -dr --prefix-addresses --show-raw-insn
+
+.*: +file format .*arm.*
+
+Disassembly of section \.text:
+0+000 <[^>]*> 0320f000 ? nopeq \{0\}
+0+004 <[^>]*> 7320f000 ? nopvc \{0\}
+0+008 <[^>]*> 7320d700 ? nopvc \{0\} ; <UNPREDICTABLE>
+
+
diff --git a/gas/testsuite/gas/arm/nops.s b/gas/testsuite/gas/arm/nops.s
new file mode 100644
index 0000000000000000000000000000000000000000..704c10b66d2e25f9748c7eca13bf38193c83d35b
--- /dev/null
+++ b/gas/testsuite/gas/arm/nops.s
@@ -0,0 +1,4 @@
+.arm
+.inst 0x0320f000
+.inst 0x7320f000
+.inst 0x7320d700
diff --git a/opcodes/arm-dis.c b/opcodes/arm-dis.c
index ab9638b9f8dda2f37732a710cdd8b3afbd0cb266..125739ba1c9a58549a20651f52cc947a10f611bd 100644
--- a/opcodes/arm-dis.c
+++ b/opcodes/arm-dis.c
@@ -1647,6 +1647,8 @@ static const struct opcode32 arm_opcodes[] =
{ARM_FEATURE_CORE_LOW (ARM_EXT_V7), 0xf57ff050, 0xfffffff0, "dmb\t%U"},
{ARM_FEATURE_CORE_LOW (ARM_EXT_V7), 0xf57ff040, 0xfffffff0, "dsb\t%U"},
{ARM_FEATURE_CORE_LOW (ARM_EXT_V7), 0xf57ff060, 0xfffffff0, "isb\t%U"},
+ {ARM_FEATURE_CORE_LOW (ARM_EXT_V7),
+ 0x0320f000, 0x0fffffff, "nop%c\t{%0-7d}"},
/* ARM V6T2 instructions. */
{ARM_FEATURE_CORE_LOW (ARM_EXT_V6T2),
@@ -2124,11 +2126,13 @@ static const struct opcode32 arm_opcodes[] =
0x01000010, 0x0fe00090, "tst%p%c\t%16-19R, %o"},
{ARM_FEATURE_CORE_LOW (ARM_EXT_V1),
- 0x03200000, 0x0fe00000, "teq%p%c\t%16-19r, %o"},
+ 0x03300000, 0x0ff00000, "teq%p%c\t%16-19r, %o"},
{ARM_FEATURE_CORE_LOW (ARM_EXT_V1),
- 0x01200000, 0x0fe00010, "teq%p%c\t%16-19r, %o"},
+ 0x01300000, 0x0ff00010, "teq%p%c\t%16-19r, %o"},
{ARM_FEATURE_CORE_LOW (ARM_EXT_V1),
- 0x01200010, 0x0fe00090, "teq%p%c\t%16-19R, %o"},
+ 0x01300010, 0x0ff00010, "teq%p%c\t%16-19R, %o"},
+ {ARM_FEATURE_CORE_LOW (ARM_EXT_V5),
+ 0x0130f000, 0x0ff0f010, "bx%c\t%0-3r"},
{ARM_FEATURE_CORE_LOW (ARM_EXT_V1),
0x03400000, 0x0fe00000, "cmp%p%c\t%16-19r, %o"},
@@ -2277,6 +2281,8 @@ static const struct opcode32 arm_opcodes[] =
0x0f000000, 0x0f000000, "svc%c\t%0-23x"},
/* The rest. */
+ {ARM_FEATURE_CORE_LOW (ARM_EXT_V7),
+ 0x03200000, 0x0fff00ff, "nop%c\t{%0-7d}" UNPREDICTABLE_INSTRUCTION},
{ARM_FEATURE_CORE_LOW (ARM_EXT_V1),
0x00000000, 0x00000000, UNDEFINED_INSTRUCTION},
{ARM_FEATURE_CORE_LOW (0),
@@ -4655,6 +4661,8 @@ print_insn_arm (bfd_vma pc, struct disassemble_info *info, long given)
if (! ARM_CPU_HAS_FEATURE (private_data->features, \
arm_ext_v6))
func (stream, "p");
+ else
+ is_unpredictable = TRUE;
}
break;
--
1.9.1
^ permalink raw reply [flat|nested] 5+ messages in thread
* Re: [binutils][objdump][ARM] Fixes for unpredictable nops and 26-bit versions of teq,tst,cmn,cmp
2015-08-05 16:58 [binutils][objdump][ARM] Fixes for unpredictable nops and 26-bit versions of teq,tst,cmn,cmp Andre Vieira
@ 2015-08-11 16:18 ` Nick Clifton
2015-08-12 9:50 ` Andre Vieira
0 siblings, 1 reply; 5+ messages in thread
From: Nick Clifton @ 2015-08-11 16:18 UTC (permalink / raw)
To: Andre Vieira, binutils
Hi Andre,
> opcodes/ChangeLog:
> 2015-07-16 Andre Vieira <andre.simoesdiasvieira@arm.com>
>
> * arm-dis.c (print_insn_arm): Disassembling for all targets V6
> and higher with ARM instruction set will now mark the 26-bit
> versions of teq,tst,cmn and cmp as UNPREDICTABLE.
> (arm_opcodes): Fix for unpredictable nop being recognized as a teq.
>
> gas/testsuite/ChangeLog:
> 2015-07-21 Andre Vieira <andre.simoesdiasvieira@arm.com>
> * gas/arm/nops.d: New.
> * gas/arm/nops.s: New.
> * gas/arm/inst.d: Changed expectation file for 26-bit teq,
> tst, cmn and cmp.
Approved - please apply.
Cheers
Nick
^ permalink raw reply [flat|nested] 5+ messages in thread
* Re: [binutils][objdump][ARM] Fixes for unpredictable nops and 26-bit versions of teq,tst,cmn,cmp
2015-08-11 16:18 ` Nick Clifton
@ 2015-08-12 9:50 ` Andre Vieira
2015-08-13 10:40 ` Nick Clifton
0 siblings, 1 reply; 5+ messages in thread
From: Andre Vieira @ 2015-08-12 9:50 UTC (permalink / raw)
To: binutils; +Cc: Nick Clifton
Hi Nick,
On 11/08/15 17:18, Nick Clifton wrote:
> Hi Andre,
>
>> opcodes/ChangeLog:
>> 2015-07-16 Andre Vieira <andre.simoesdiasvieira@arm.com>
>>
>> * arm-dis.c (print_insn_arm): Disassembling for all targets V6
>> and higher with ARM instruction set will now mark the 26-bit
>> versions of teq,tst,cmn and cmp as UNPREDICTABLE.
>> (arm_opcodes): Fix for unpredictable nop being recognized as a teq.
>>
>> gas/testsuite/ChangeLog:
>> 2015-07-21 Andre Vieira <andre.simoesdiasvieira@arm.com>
>> * gas/arm/nops.d: New.
>> * gas/arm/nops.s: New.
>> * gas/arm/inst.d: Changed expectation file for 26-bit teq,
>> tst, cmn and cmp.
>
> Approved - please apply.
>
> Cheers
> Nick
>
I don't think I have write permissions.
Cheers,
Andre
^ permalink raw reply [flat|nested] 5+ messages in thread
* Re: [binutils][objdump][ARM] Fixes for unpredictable nops and 26-bit versions of teq,tst,cmn,cmp
2015-08-12 9:50 ` Andre Vieira
@ 2015-08-13 10:40 ` Nick Clifton
2015-09-04 9:12 ` Andre Vieira
0 siblings, 1 reply; 5+ messages in thread
From: Nick Clifton @ 2015-08-13 10:40 UTC (permalink / raw)
To: Andre Vieira; +Cc: binutils
Hi Andre,
> I don't think I have write permissions.
No worries - I have checked it in for you.
Cheers
Nick
^ permalink raw reply [flat|nested] 5+ messages in thread
* Re: [binutils][objdump][ARM] Fixes for unpredictable nops and 26-bit versions of teq,tst,cmn,cmp
2015-08-13 10:40 ` Nick Clifton
@ 2015-09-04 9:12 ` Andre Vieira
0 siblings, 0 replies; 5+ messages in thread
From: Andre Vieira @ 2015-09-04 9:12 UTC (permalink / raw)
To: binutils
On 13/08/15 11:40, Nick Clifton wrote:
> Hi Andre,
>
>> I don't think I have write permissions.
>
> No worries - I have checked it in for you.
>
> Cheers
> Nick
>
>
Hello,
I would like to backport this patch to 2.24 and 2.25. Are there any
objections?
BR,
Andre
^ permalink raw reply [flat|nested] 5+ messages in thread
end of thread, other threads:[~2015-09-04 9:12 UTC | newest]
Thread overview: 5+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2015-08-05 16:58 [binutils][objdump][ARM] Fixes for unpredictable nops and 26-bit versions of teq,tst,cmn,cmp Andre Vieira
2015-08-11 16:18 ` Nick Clifton
2015-08-12 9:50 ` Andre Vieira
2015-08-13 10:40 ` Nick Clifton
2015-09-04 9:12 ` Andre Vieira
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for read-only IMAP folder(s) and NNTP newsgroup(s).