[binutils-gdb] Fix SBO bit in disassembly mask for ldrah on AArch64.

Tamar Christina tnfchris@sourceware.org
Fri Jul 6 15:23:00 GMT 2018


https://sourceware.org/git/gitweb.cgi?p=binutils-gdb.git;h=f311ba7ed84d66ae2cd77bd969747d7ab959d866

commit f311ba7ed84d66ae2cd77bd969747d7ab959d866
Author: Tamar Christina <tamar.christina@arm.com>
Date:   Fri Jul 6 16:18:19 2018 +0100

    Fix SBO bit in disassembly mask for ldrah on AArch64.
    
    The disassembly mask for ldarh incorrectly didn't mask out bit 20 which
    is part of the SBO part of the instruction and shouldn't be considered input.
    
    This fixes the wrong bit fixing the disassembly of instructions to
    ldarh and makes the behavior consistent.
    
    opcodes/
    
    	PR binutils/23242
    	* aarch64-tbl.h (ldarh): Fix disassembly mask.

Diff:
---
 opcodes/ChangeLog     | 5 +++++
 opcodes/aarch64-tbl.h | 2 +-
 2 files changed, 6 insertions(+), 1 deletion(-)

diff --git a/opcodes/ChangeLog b/opcodes/ChangeLog
index 70d35eb..3b92887 100644
--- a/opcodes/ChangeLog
+++ b/opcodes/ChangeLog
@@ -1,5 +1,10 @@
 2018-07-06  Tamar Christina  <tamar.christina@arm.com>
 
+	PR binutils/23242
+	* aarch64-tbl.h (ldarh): Fix disassembly mask.
+
+2018-07-06  Tamar Christina  <tamar.christina@arm.com>
+
 	PR binutils/23369
 	* aarch64-opc.c (aarch64_sys_regs): Make read/write csselr_el1,
 	vsesr_el2, osdtrrx_el1, osdtrtx_el1, pmsidr_el1.
diff --git a/opcodes/aarch64-tbl.h b/opcodes/aarch64-tbl.h
index c720fea..559efdb 100644
--- a/opcodes/aarch64-tbl.h
+++ b/opcodes/aarch64-tbl.h
@@ -3219,7 +3219,7 @@ struct aarch64_opcode aarch64_opcode_table[] =
   CORE_INSN ("ldxrh", 0x485f7c00, 0xffe08000, ldstexcl, 0, OP2 (Rt, ADDR_SIMPLE), QL_W1_LDST_EXC, 0),
   CORE_INSN ("ldaxrh", 0x485ffc00, 0xffe08000, ldstexcl, 0, OP2 (Rt, ADDR_SIMPLE), QL_W1_LDST_EXC, 0),
   CORE_INSN ("stlrh", 0x489ffc00, 0xffe08000, ldstexcl, 0, OP2 (Rt, ADDR_SIMPLE), QL_W1_LDST_EXC, 0),
-  CORE_INSN ("ldarh", 0x48dffc00, 0xffeffc00, ldstexcl, 0, OP2 (Rt, ADDR_SIMPLE), QL_W1_LDST_EXC, 0),
+  CORE_INSN ("ldarh", 0x48dffc00, 0xfffffc00, ldstexcl, 0, OP2 (Rt, ADDR_SIMPLE), QL_W1_LDST_EXC, 0),
   CORE_INSN ("stxr", 0x88007c00, 0xbfe08000, ldstexcl, 0, OP3 (Rs, Rt, ADDR_SIMPLE), QL_R2_LDST_EXC, F_GPRSIZE_IN_Q),
   CORE_INSN ("stlxr", 0x8800fc00, 0xbfe08000, ldstexcl, 0, OP3 (Rs, Rt, ADDR_SIMPLE), QL_R2_LDST_EXC, F_GPRSIZE_IN_Q),
   CORE_INSN ("stxp", 0x88200000, 0xbfe08000, ldstexcl, 0, OP4 (Rs, Rt, Rt2, ADDR_SIMPLE), QL_R3_LDST_EXC, F_GPRSIZE_IN_Q),



More information about the Binutils-cvs mailing list