[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