diff options
| author | Andreas Arnez <arnez@linux.vnet.ibm.com> | 2014-08-19 15:42:13 +0100 |
|---|---|---|
| committer | Nick Clifton <nickc@redhat.com> | 2014-08-19 15:42:13 +0100 |
| commit | 9ace48f3d7d80ce09c5df60cccb433470410b11b (patch) | |
| tree | 3da3986f32033753fb8ca3bd51aaecbba9ddeb43 /opcodes/ChangeLog | |
| parent | cb417230fb10c4f5b97cf8cffa973937f7bc8b6c (diff) | |
| download | gdb-9ace48f3d7d80ce09c5df60cccb433470410b11b.zip gdb-9ace48f3d7d80ce09c5df60cccb433470410b11b.tar.gz gdb-9ace48f3d7d80ce09c5df60cccb433470410b11b.tar.bz2 | |
This patch set mainly aims at improving the S/390 disassembler's
readability and also fixes some minor issues.
S/390: Split disassembler routine into smaller functions
S/390: Fix disassembler's treatment of signed/unsigned operands
S/390: Fix off-by-one error in disassembler initialization
S/390: Simplify opcode search loop in disassembler
S/390: Drop function pointer dereferences in disassembler
S/390: Various minor simplifications in disassembler
Diffstat (limited to 'opcodes/ChangeLog')
| -rw-r--r-- | opcodes/ChangeLog | 27 |
1 files changed, 27 insertions, 0 deletions
diff --git a/opcodes/ChangeLog b/opcodes/ChangeLog index 0e8e850..82ec520 100644 --- a/opcodes/ChangeLog +++ b/opcodes/ChangeLog @@ -1,3 +1,30 @@ +2014-08-19 Andreas Arnez <arnez@linux.vnet.ibm.com> + + * s390-dis.c (s390_insn_length, s390_insn_matches_opcode) + (s390_print_insn_with_opcode, opcode_mask_more_specific): New + static functions, code was moved from... + (print_insn_s390): ...here. + (s390_extract_operand): Adjust comment. Change type of first + parameter from 'unsigned char *' to 'const bfd_byte *'. + (union operand_value): New. + (s390_extract_operand): Change return type to union operand_value. + Also avoid integer overflow in sign-extension. + (s390_print_insn_with_opcode): Adjust to changed return value from + s390_extract_operand(). Change "%i" printf format to "%u" for + unsigned values. + (init_disasm): Simplify initialization of opc_index[]. This also + fixes an access after the last element of s390_opcodes[]. + (print_insn_s390): Simplify the opcode search loop. + Check architecture mask against all searched opcodes, not just the + first matching one. + (s390_print_insn_with_opcode): Drop function pointer dereferences + without effect. + (print_insn_s390): Likewise. + (s390_insn_length): Simplify formula for return value. + (s390_print_insn_with_opcode): Avoid special handling for the + separator before the first operand. Use new local variable + 'flags' in place of 'operand->flags'. + 2014-08-14 Mike Frysinger <vapier@gentoo.org> * bfin-dis.c (struct private): Change int's to bfd_boolean's. |