diff options
| author | Alan Modra <amodra@bigpond.net.au> | 2003-06-30 04:09:27 +0000 |
|---|---|---|
| committer | Alan Modra <amodra@bigpond.net.au> | 2003-06-30 04:09:27 +0000 |
| commit | 26bc3b06b7a97daa63d39ae66cc8ae9e909e76cd (patch) | |
| tree | 7fa5b71cae62a3162ebcd365e76d683bb81b04b3 | |
| parent | 3afbf55cf3d97ed4be5cd493cfba671c12bd7016 (diff) | |
| download | gdb-26bc3b06b7a97daa63d39ae66cc8ae9e909e76cd.tar.gz | |
* cpu-ia64-opc.c (ext_imms_scaled): Don't sign extend using shifts.
(ins_imms, ins_immsm1u4): Likewise. Warning fix.
| -rw-r--r-- | bfd/ChangeLog | 5 | ||||
| -rw-r--r-- | bfd/cpu-ia64-opc.c | 18 |
2 files changed, 11 insertions, 12 deletions
diff --git a/bfd/ChangeLog b/bfd/ChangeLog index f2e5665b982..47e0038fbf1 100644 --- a/bfd/ChangeLog +++ b/bfd/ChangeLog @@ -1,3 +1,8 @@ +2003-06-30 Alan Modra <amodra@bigpond.net.au> + + * cpu-ia64-opc.c (ext_imms_scaled): Don't sign extend using shifts. + (ins_imms, ins_immsm1u4): Likewise. Warning fix. + 2003-06-29 Alan Modra <amodra@bigpond.net.au> * archive.c: Convert to C90, remove unnecessary prototypes and casts. diff --git a/bfd/cpu-ia64-opc.c b/bfd/cpu-ia64-opc.c index 1025256490e..3cafb9f14d9 100644 --- a/bfd/cpu-ia64-opc.c +++ b/bfd/cpu-ia64-opc.c @@ -161,8 +161,8 @@ static const char* ext_imms_scaled (const struct ia64_operand *self, ia64_insn code, ia64_insn *valuep, int scale) { - int i, bits = 0, total = 0, shift; - BFD_HOST_64_BIT val = 0; + int i, bits = 0, total = 0; + BFD_HOST_64_BIT val = 0, sign; for (i = 0; i < NELEMS (self->field) && self->field[i].bits; ++i) { @@ -172,8 +172,8 @@ ext_imms_scaled (const struct ia64_operand *self, ia64_insn code, total += bits; } /* sign extend: */ - shift = 8*sizeof (val) - total; - val = (val << shift) >> shift; + sign = (BFD_HOST_64_BIT) 1 << (total - 1); + val = (val ^ sign) - sign; *valuep = (val << scale); return 0; @@ -188,10 +188,7 @@ ins_imms (const struct ia64_operand *self, ia64_insn value, ia64_insn *code) static const char* ins_immsu4 (const struct ia64_operand *self, ia64_insn value, ia64_insn *code) { - if (value == (BFD_HOST_U_64_BIT) 0x100000000) - value = 0; - else - value = (((BFD_HOST_64_BIT)value << 32) >> 32); + value = ((value & 0xffffffff) ^ 0x80000000) - 0x80000000; return ins_imms_scaled (self, value, code, 0); } @@ -213,10 +210,7 @@ static const char* ins_immsm1u4 (const struct ia64_operand *self, ia64_insn value, ia64_insn *code) { - if (value == (BFD_HOST_U_64_BIT) 0x100000000) - value = 0; - else - value = (((BFD_HOST_64_BIT)value << 32) >> 32); + value = ((value & 0xffffffff) ^ 0x80000000) - 0x80000000; --value; return ins_imms_scaled (self, value, code, 0); |