espressif/binutils-gdb
1
0
Fork 0
mirror of https://github.com/espressif/binutils-gdb.git synced 2025-06-22 19:09:31 +08:00
Code Issues Packages Projects Releases Wiki Activity

MIPS16: Remove unused `>' operand code

Browse Source 
This code has never been used throughout the repository history, and
likely not before either, as due to the assymetry of MIPS16 instruction
set encoding there are no 32-bit shift operations having their immediate
shift count placed in the position of the usual `rx' instruction field.

	gas/
	* config/tc-mips.c (mips16_macro_build) <'>'>: Remove case.

	include/
	* opcode/mips.h: Remove references to `>' operand code.

	opcodes/
	* mips16-opc.c (decode_mips16_operand) <'>'>: Remove cases.
This commit is contained in:
Maciej W. Rozycki
2016-12-09 23:11:40 +00:00
parent f17ecb4bac
commit 64c1118340
6 changed files with 13 additions and 5 deletions
Download Patch File Download Diff File Expand all files Collapse all files

4
gas/ChangeLog
View File

@ -1,3 +1,7 @@
2016-12-09 Maciej W. Rozycki <macro@imgtec.com>
* config/tc-mips.c (mips16_macro_build) <'>'>: Remove case.
2016-12-09 Maciej W. Rozycki <macro@imgtec.com> 2016-12-09 Maciej W. Rozycki <macro@imgtec.com>
* testsuite/gas/mips/mips16-extend.d: New test. * testsuite/gas/mips/mips16-extend.d: New test.

1
gas/config/tc-mips.c
View File

@ -8723,7 +8723,6 @@ mips16_macro_build (expressionS *ep, const char *name, const char *fmt,
break; break;
case '<': case '<':
case '>':
case '4': case '4':
case '5': case '5':
case 'H': case 'H':

4
include/ChangeLog
View File

@ -1,3 +1,7 @@
2016-12-09 Maciej W. Rozycki <macro@imgtec.com>
* opcode/mips.h: Remove references to `>' operand code.
2016-12-07 Maciej W. Rozycki <macro@imgtec.com> 2016-12-07 Maciej W. Rozycki <macro@imgtec.com>
* opcode/mips.h (INSN_CHIP_MASK): Update according to bit use. * opcode/mips.h (INSN_CHIP_MASK): Update according to bit use.

3
include/opcode/mips.h
View File

@ -1809,7 +1809,6 @@ extern int bfd_mips_num_opcodes;
The remaining codes may be extended. Except as otherwise noted, The remaining codes may be extended. Except as otherwise noted,
the full extended operand is a 16 bit signed value. the full extended operand is a 16 bit signed value.
"<" 3 bit unsigned shift count * 0 (MIPS16OP_*_RZ) (full 5 bit unsigned) "<" 3 bit unsigned shift count * 0 (MIPS16OP_*_RZ) (full 5 bit unsigned)
">" 3 bit unsigned shift count * 0 (MIPS16OP_*_RX) (full 5 bit unsigned)
"[" 3 bit unsigned shift count * 0 (MIPS16OP_*_RZ) (full 6 bit unsigned) "[" 3 bit unsigned shift count * 0 (MIPS16OP_*_RZ) (full 6 bit unsigned)
"]" 3 bit unsigned shift count * 0 (MIPS16OP_*_RX) (full 6 bit unsigned) "]" 3 bit unsigned shift count * 0 (MIPS16OP_*_RX) (full 6 bit unsigned)
"4" 4 bit signed immediate * 0 (MIPS16OP_*_IMM4) (full 15 bit signed) "4" 4 bit signed immediate * 0 (MIPS16OP_*_IMM4) (full 15 bit signed)
@ -1834,7 +1833,7 @@ extern int bfd_mips_num_opcodes;
Characters used so far, for quick reference when adding more: Characters used so far, for quick reference when adding more:
" 456 8 0" " 456 8 0"
"[]<>" "[]<"
"ABCDE HI KLM P RS UVWXYZ" "ABCDE HI KLM P RS UVWXYZ"
"a e ijklm pq vwxyz" "a e ijklm pq vwxyz"
*/ */

4
opcodes/ChangeLog
View File

@ -1,3 +1,7 @@
2016-12-09 Maciej W. Rozycki <macro@imgtec.com>
* mips16-opc.c (decode_mips16_operand) <'>'>: Remove cases.
2016-12-09 Maciej W. Rozycki <macro@imgtec.com> 2016-12-09 Maciej W. Rozycki <macro@imgtec.com>
* mips16-opc.c (decode_mips16_operand) <'e'>: Use HINT rather * mips16-opc.c (decode_mips16_operand) <'e'>: Use HINT rather

2
opcodes/mips16-opc.c
View File

@ -76,7 +76,6 @@ decode_mips16_operand (char type, bfd_boolean extended_p)
switch (type) switch (type)
{ {
case '<': UINT (5, 0); case '<': UINT (5, 0);
case '>': UINT (5, 0);
case '[': UINT (6, 0); case '[': UINT (6, 0);
case ']': UINT (6, 0); case ']': UINT (6, 0);
@ -105,7 +104,6 @@ decode_mips16_operand (char type, bfd_boolean extended_p)
switch (type) switch (type)
{ {
case '<': INT_ADJ (3, 2, 8, 0, FALSE); case '<': INT_ADJ (3, 2, 8, 0, FALSE);
case '>': INT_ADJ (3, 8, 8, 0, FALSE);
case '[': INT_ADJ (3, 2, 8, 0, FALSE); case '[': INT_ADJ (3, 2, 8, 0, FALSE);
case ']': INT_ADJ (3, 8, 8, 0, FALSE); case ']': INT_ADJ (3, 8, 8, 0, FALSE);