ubsan: ns32k: left shift cannot be represented in type 'int'

	* ns32k-dis.c (bit_extract): Use unsigned arithmetic.
	(bit_extract_simple, sign_extend): Likewise.

diff --git a/opcodes/ChangeLog b/opcodes/ChangeLog
index b36a9e203614111e46165a5c0dca9bc1a20bce46..ddf8cd0072489ffcbfc285a0cac4360d21557a93 100644 (file)
--- a/opcodes/ChangeLog
+++ b/opcodes/ChangeLog
@@ -1,3 +1,8 @@
+2019-12-11  Alan Modra  <amodra@gmail.com>
+
+       * ns32k-dis.c (bit_extract): Use unsigned arithmetic.
+       (bit_extract_simple, sign_extend): Likewise.
+
 2019-12-11  Alan Modra  <amodra@gmail.com>
 
        * nios2-dis.c (nios2_print_insn_arg): Use 1u << 31.

diff --git a/opcodes/ns32k-dis.c b/opcodes/ns32k-dis.c
index 22a9389ecff2dee701885b045e063c9c5b4fdb2a..5e6f0964b7aa626796f64dab47cfbdc2483ce48d 100644 (file)
--- a/opcodes/ns32k-dis.c
+++ b/opcodes/ns32k-dis.c
@@ -262,8 +262,8 @@ list_search (int reg_value, const struct ns32k_option *optionP, char *result)
 static int
 bit_extract (bfd_byte *buffer, int offset, int count)
 {
-  int result;
-  int bit;
+  unsigned int result;
+  unsigned int bit;
 
   if (offset < 0 || count < 0)
     return 0;
@@ -291,8 +291,8 @@ bit_extract (bfd_byte *buffer, int offset, int count)
 static int
 bit_extract_simple (bfd_byte *buffer, int offset, int count)
 {
-  int result;
-  int bit;
+  unsigned int result;
+  unsigned int bit;
 
   if (offset < 0 || count < 0)
     return 0;
@@ -325,12 +325,10 @@ bit_copy (bfd_byte *buffer, int offset, int count, char *to)
 }
 
 static int
-sign_extend (int value, int bits)
+sign_extend (unsigned int value, unsigned int bits)
 {
-  value = value & ((1 << bits) - 1);
-  return (value & (1 << (bits - 1))
-         ? value | (~((1 << bits) - 1))
-         : value);
+  unsigned int sign = 1u << bits;
+  return ((value & (sign - 1)) ^ sign) - sign;
 }
 
 static void