ubsan: moxie: left shift of negative value

Commit 8c9b4171877df didn't remove a glaring left shift of a number
that had just been sign extended.

	* moxie-dis.c (INST2OFFSET): Don't left shift a signed value.
	(print_insn_moxie): Remove unnecessary cast.

diff --git a/opcodes/ChangeLog b/opcodes/ChangeLog
index 9e0d57b7c9c6ac183596edc371108f637f45a1a8..2d1c9d3f7d4eec49ccf0f32bfeb7a36596336cf0 100644 (file)
--- a/opcodes/ChangeLog
+++ b/opcodes/ChangeLog
@@ -1,3 +1,8 @@
+2019-12-16  Alan Modra  <amodra@gmail.com>
+
+       * moxie-dis.c (INST2OFFSET): Don't left shift a signed value.
+       (print_insn_moxie): Remove unnecessary cast.
+
 2019-12-12  Alan Modra  <amodra@gmail.com>
 
        * csky-dis.c (csky_chars_to_number): Remove abort and unnecessary

diff --git a/opcodes/moxie-dis.c b/opcodes/moxie-dis.c
index cbfcf958806831bcd32b85eca1fdb9f891e6e65b..1d06e184794217a48b7db9f1aa17112abb727799 100644 (file)
--- a/opcodes/moxie-dis.c
+++ b/opcodes/moxie-dis.c
@@ -33,7 +33,7 @@ static void *stream;
 /* Macros to extract operands from the instruction word.  */
 #define OP_A(i) ((i >> 4) & 0xf)
 #define OP_B(i) (i & 0xf)
-#define INST2OFFSET(o) (((((o) & 0x3ff) ^ 0x200) - 0x200) << 1)
+#define INST2OFFSET(o) (((((o) & 0x3ff) ^ 0x200) - 0x200) * 2)
 
 static const char * reg_names[16] =
   { "$fp", "$sp", "$r0", "$r1", "$r2", "$r3", "$r4", "$r5",
@@ -210,8 +210,7 @@ print_insn_moxie (bfd_vma addr, struct disassemble_info * info)
        {
        case MOXIE_F3_PCREL:
          fpr (stream, "%s\t", opcode->name);
-         info->print_address_func ((bfd_vma) (addr + INST2OFFSET(iword) + 2),
-                                   info);
+         info->print_address_func (addr + INST2OFFSET (iword) + 2, info);
          break;
         case MOXIE_BAD:
          fpr (stream, "bad");