aboutsummaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorHenry Wong <henry@stuffedcow.net>2018-02-12 14:50:42 +0000
committerMaciej W. Rozycki <macro@mips.com>2018-02-12 14:50:42 +0000
commitd2159fdc0f0ac1d0aaafab725b930e78a8793494 (patch)
tree8003b1ce5358bf26c01eba1de7ee8215f5cc2bee
parent830db0485e19000985ccfdbacda4d4d5d62583bb (diff)
downloadgdb-d2159fdc0f0ac1d0aaafab725b930e78a8793494.zip
gdb-d2159fdc0f0ac1d0aaafab725b930e78a8793494.tar.gz
gdb-d2159fdc0f0ac1d0aaafab725b930e78a8793494.tar.bz2
MIPS: Fix encoding for MIPSr6 sigrie instruction.
The instruction encoding for the MIPS r6 sigrie instruction seems to be incorrect. It's currently 0x4170xxxx (which overlaps with ei, di, evp, and dvp), but should be 0x0417xxxx. See ISA reference[1][2]. References: [1] "MIPS Architecture for Programmers Volume II-A: The MIPS32 Instruction Set Manual", Imagination Technologies, Inc., Document Number: MD00086, Revision 6.06, December 15, 2016, Table A.4 "MIPS32 REGIMM Encoding of rt Field", p. 452 [2] "MIPS Architecture For Programmers Volume II-A: The MIPS64 Instruction Set Reference Manual", Imagination Technologies, Inc., Document Number: MD00087, Revision 6.06, December 15, 2016, Table A.4 "MIPS64 REGIMM Encoding of rt Field", p. 581 opcodes/ * mips-opc.c (mips_builtin_opcodes): Correct "sigrie" encoding. gas/ * testsuite/gas/mips/r6.d: Update for "sigrie" encoding fix. * testsuite/gas/mips/r6-n32.d: Likewise. * testsuite/gas/mips/r6-n64.d: Likewise.
-rw-r--r--gas/ChangeLog6
-rw-r--r--gas/testsuite/gas/mips/r6-n32.d4
-rw-r--r--gas/testsuite/gas/mips/r6-n64.d4
-rw-r--r--gas/testsuite/gas/mips/r6.d4
-rw-r--r--opcodes/ChangeLog4
-rw-r--r--opcodes/mips-opc.c2
6 files changed, 17 insertions, 7 deletions
diff --git a/gas/ChangeLog b/gas/ChangeLog
index 87b5073..dfb5edf 100644
--- a/gas/ChangeLog
+++ b/gas/ChangeLog
@@ -1,3 +1,9 @@
+2018-02-12 Henry Wong <henry@stuffedcow.net>
+
+ * testsuite/gas/mips/r6.d: Update for "sigrie" encoding fix.
+ * testsuite/gas/mips/r6-n32.d: Likewise.
+ * testsuite/gas/mips/r6-n64.d: Likewise.
+
2018-02-12 Nick Clifton <nickc@redhat.com>
* po/ru.po: Updated Russian translation.
diff --git a/gas/testsuite/gas/mips/r6-n32.d b/gas/testsuite/gas/mips/r6-n32.d
index fb55086..c9efa1d 100644
--- a/gas/testsuite/gas/mips/r6-n32.d
+++ b/gas/testsuite/gas/mips/r6-n32.d
@@ -497,6 +497,6 @@ Disassembly of section .text:
0+0598 <[^>]*> 41600024 dvp
0+059c <[^>]*> 41620004 evp v0
0+05a0 <[^>]*> 41620024 dvp v0
-0+05a4 <[^>]*> 41700000 sigrie 0x0
-0+05a8 <[^>]*> 4170ffff sigrie 0xffff
+0+05a4 <[^>]*> 04170000 sigrie 0x0
+0+05a8 <[^>]*> 0417ffff sigrie 0xffff
\.\.\.
diff --git a/gas/testsuite/gas/mips/r6-n64.d b/gas/testsuite/gas/mips/r6-n64.d
index fd4da21..fa0e86b 100644
--- a/gas/testsuite/gas/mips/r6-n64.d
+++ b/gas/testsuite/gas/mips/r6-n64.d
@@ -753,6 +753,6 @@ Disassembly of section .text:
0+0598 <[^>]*> 41600024 dvp
0+059c <[^>]*> 41620004 evp v0
0+05a0 <[^>]*> 41620024 dvp v0
-0+05a4 <[^>]*> 41700000 sigrie 0x0
-0+05a8 <[^>]*> 4170ffff sigrie 0xffff
+0+05a4 <[^>]*> 04170000 sigrie 0x0
+0+05a8 <[^>]*> 0417ffff sigrie 0xffff
\.\.\.
diff --git a/gas/testsuite/gas/mips/r6.d b/gas/testsuite/gas/mips/r6.d
index 8588e92..9faa478 100644
--- a/gas/testsuite/gas/mips/r6.d
+++ b/gas/testsuite/gas/mips/r6.d
@@ -496,6 +496,6 @@ Disassembly of section .text:
0+0598 <[^>]*> 41600024 dvp
0+059c <[^>]*> 41620004 evp v0
0+05a0 <[^>]*> 41620024 dvp v0
-0+05a4 <[^>]*> 41700000 sigrie 0x0
-0+05a8 <[^>]*> 4170ffff sigrie 0xffff
+0+05a4 <[^>]*> 04170000 sigrie 0x0
+0+05a8 <[^>]*> 0417ffff sigrie 0xffff
\.\.\.
diff --git a/opcodes/ChangeLog b/opcodes/ChangeLog
index 981f963..c4f9039 100644
--- a/opcodes/ChangeLog
+++ b/opcodes/ChangeLog
@@ -1,3 +1,7 @@
+2018-02-12 Henry Wong <henry@stuffedcow.net>
+
+ * mips-opc.c (mips_builtin_opcodes): Correct "sigrie" encoding.
+
2018-02-05 Nick Clifton <nickc@redhat.com>
* po/pt_BR.po: Updated Brazilian Portuguese translation.
diff --git a/opcodes/mips-opc.c b/opcodes/mips-opc.c
index 180d613..b0c6195 100644
--- a/opcodes/mips-opc.c
+++ b/opcodes/mips-opc.c
@@ -1867,7 +1867,7 @@ const struct mips_opcode mips_builtin_opcodes[] =
{"shfl.repa.qh", "X,Y,Z", 0x7b20001f, 0xffe0003f, WR_1|RD_2|RD_3|FP_D, 0, 0, MX, 0 },
{"shfl.repb.qh", "X,Y,Z", 0x7ba0001f, 0xffe0003f, WR_1|RD_2|RD_3|FP_D, 0, 0, MX, 0 },
{"shfl.upsl.ob", "X,Y,Z", 0x78c0001f, 0xffe0003f, WR_1|RD_2|RD_3|FP_D, 0, SB1, MX, 0 },
-{"sigrie", "u", 0x41700000, 0xffff0000, TRAP, 0, I37, 0, 0 },
+{"sigrie", "u", 0x04170000, 0xffff0000, TRAP, 0, I37, 0, 0 },
{"sle", "d,v,t", 0, (int) M_SLE, INSN_MACRO, 0, I1, 0, 0 },
{"sle", "d,v,I", 0, (int) M_SLE_I, INSN_MACRO, 0, I1, 0, 0 },
{"sle", "S,T", 0x46a0003e, 0xffe007ff, RD_1|RD_2|WR_CC|FP_D, 0, IL2E, 0, 0 },