This is the mail archive of the
binutils@sourceware.org
mailing list for the binutils project.
[committed] Remove unused 'Y' case from mips16_macro_build
- From: Richard Sandiford <rdsandiford at googlemail dot com>
- To: binutils at sourceware dot org
- Date: Sun, 14 Jul 2013 14:19:03 +0100
- Subject: [committed] Remove unused 'Y' case from mips16_macro_build
I was confused as to why the 'Y' case in mips16_macro_build swizzles
its argument, whereas all other macro_build and mips16_macro_build
arguments are preformated. It seems to be dead code though, so this
patch removes it.
Tested on various targets and applied.
Richard
gas/
* config/tc-mips.c (mips16_macro_build): Remove 'Y' case.
Index: gas/config/tc-mips.c
===================================================================
--- gas/config/tc-mips.c 2013-07-14 10:31:27.869873880 +0100
+++ gas/config/tc-mips.c 2013-07-14 10:32:15.477331822 +0100
@@ -5688,16 +5688,6 @@ mips16_macro_build (expressionS *ep, con
MIPS16_INSERT_OPERAND (REGR32, insn, va_arg (*args, int));
continue;
- case 'Y':
- {
- int regno;
-
- regno = va_arg (*args, int);
- regno = ((regno & 7) << 2) | ((regno & 0x18) >> 3);
- MIPS16_INSERT_OPERAND (REG32R, insn, regno);
- }
- continue;
-
case '<':
case '>':
case '4':