[binutils-gdb] csky: tidy csky_chars_to_number

Alan Modra amodra@sourceware.org
Thu Dec 12 11:07:00 GMT 2019


https://sourceware.org/git/gitweb.cgi?p=binutils-gdb.git;h=967354c3b9b0306785dde600ab6f851fe1418612

commit 967354c3b9b0306785dde600ab6f851fe1418612
Author: Alan Modra <amodra@gmail.com>
Date:   Thu Dec 12 16:44:24 2019 +1030

    csky: tidy csky_chars_to_number
    
    	* csky-dis.c (csky_chars_to_number): Remove abort and unnecessary
    	mask.

Diff:
---
 opcodes/ChangeLog  | 5 +++++
 opcodes/csky-dis.c | 6 ++----
 2 files changed, 7 insertions(+), 4 deletions(-)

diff --git a/opcodes/ChangeLog b/opcodes/ChangeLog
index 2a44855..9e0d57b 100644
--- a/opcodes/ChangeLog
+++ b/opcodes/ChangeLog
@@ -1,3 +1,8 @@
+2019-12-12  Alan Modra  <amodra@gmail.com>
+
+	* csky-dis.c (csky_chars_to_number): Remove abort and unnecessary
+	mask.
+
 2019-12-11  Alan Modra  <amodra@gmail.com>
 
 	* arc-dis.c (BITS): Don't truncate high bits with shifts.
diff --git a/opcodes/csky-dis.c b/opcodes/csky-dis.c
index af830f3..d7ffb13 100644
--- a/opcodes/csky-dis.c
+++ b/opcodes/csky-dis.c
@@ -134,17 +134,15 @@ csky_get_mask (struct csky_opcode_info const *pinfo)
 static unsigned int
 csky_chars_to_number (unsigned char * buf, int n)
 {
-  if (n <= 0)
-    abort ();
   int i;
   unsigned int val = 0;
 
   if (dis_info.info->endian == BFD_ENDIAN_BIG)
     for (i = 0; i < n; i++)
-      val = val << 8 | (buf[i] & 0xff);
+      val = val << 8 | buf[i];
   else
     for (i = n - 1; i >= 0; i--)
-      val = val << 8 | (buf[i] & 0xff);
+      val = val << 8 | buf[i];
   return val;
 }



More information about the Binutils-cvs mailing list