From: Frank Ch. Eigler Date: Mon, 7 May 2001 17:55:21 +0000 (+0000) Subject: * cgen asm/disasm X-Git-Url: http://drtracing.org/?a=commitdiff_plain;h=a00ad97d613fec56fab39a83da4205bffc9dc4cc;p=deliverable%2Fbinutils-gdb.git * cgen asm/disasm [opcodes/ChangeLog] 2001-05-07 Frank Ch. Eigler * cgen-dis.in (default_print_insn): Tolerate min * iformat.scm (compute-insn-base-mask-length): Rewrite to tolerate various-base-length instruction sets. --- diff --git a/opcodes/ChangeLog b/opcodes/ChangeLog index f4247e96c6..d39051533a 100644 --- a/opcodes/ChangeLog +++ b/opcodes/ChangeLog @@ -1,3 +1,10 @@ +2001-05-07 Frank Ch. Eigler + + * cgen-dis.in (default_print_insn): Tolerate min * disassemble.c (disassembler_usage): Remove unused attribute. diff --git a/opcodes/cgen-dis.in b/opcodes/cgen-dis.in index b2865f8cba..91fed7b0a7 100644 --- a/opcodes/cgen-dis.in +++ b/opcodes/cgen-dis.in @@ -334,18 +334,27 @@ default_print_insn (cd, pc, info) disassemble_info *info; { char buf[CGEN_MAX_INSN_SIZE]; + int buflen; int status; - /* Read the base part of the insn. */ + /* Attempt to read the base part of the insn. */ + buflen = cd->base_insn_bitsize / 8; + status = (*info->read_memory_func) (pc, buf, buflen, info); + + /* Try again with the minimum part, if min < base. */ + if (status != 0 && (cd->min_insn_bitsize < cd->base_insn_bitsize)) + { + buflen = cd->min_insn_bitsize / 8; + status = (*info->read_memory_func) (pc, buf, buflen, info); + } - status = (*info->read_memory_func) (pc, buf, cd->base_insn_bitsize / 8, info); if (status != 0) { (*info->memory_error_func) (status, pc, info); return -1; } - return print_insn (cd, pc, info, buf, cd->base_insn_bitsize / 8); + return print_insn (cd, pc, info, buf, buflen); } /* Main entry point. diff --git a/opcodes/cgen-ibld.in b/opcodes/cgen-ibld.in index 528e60979c..ca2f1e5fcc 100644 --- a/opcodes/cgen-ibld.in +++ b/opcodes/cgen-ibld.in @@ -428,9 +428,9 @@ extract_normal (cd, ex_info, insn_value, attrs, word_offset, start, length, word_length = total_length; } - /* Does the value reside in INSN_VALUE? */ + /* Does the value reside in INSN_VALUE, and at the right alignment? */ - if (CGEN_INT_INSN_P || word_offset == 0) + if (CGEN_INT_INSN_P || (word_offset == 0 && word_length == total_length)) { if (CGEN_INSN_LSB0_P) value = insn_value >> ((word_offset + start + 1) - length);