X-Git-Url: http://drtracing.org/?a=blobdiff_plain;f=opcodes%2Fcgen-asm.c;h=b1675045d9d1139cb850b54657796f375d05a84d;hb=50d036364fb2a71b3ac9a0b0cdbe58296832a1b2;hp=e522d160157e116f4fea1264939a1d3655f2830e;hpb=e98fe4f7b54cbdf29aef9287bbb1bea8801dd05a;p=deliverable%2Fbinutils-gdb.git diff --git a/opcodes/cgen-asm.c b/opcodes/cgen-asm.c index e522d16015..b1675045d9 100644 --- a/opcodes/cgen-asm.c +++ b/opcodes/cgen-asm.c @@ -1,78 +1,60 @@ /* CGEN generic assembler support code. + Copyright (C) 1996-2020 Free Software Foundation, Inc. - Copyright (C) 1996, 1997, 1998 Free Software Foundation, Inc. + This file is part of libopcodes. - This file is part of the GNU Binutils and GDB, the GNU debugger. - - This program is free software; you can redistribute it and/or modify + This library is free software; you can redistribute it and/or modify it under the terms of the GNU General Public License as published by - the Free Software Foundation; either version 2, or (at your option) + the Free Software Foundation; either version 3, or (at your option) any later version. - This program is distributed in the hope that it will be useful, - but WITHOUT ANY WARRANTY; without even the implied warranty of - MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the - GNU General Public License for more details. + It is distributed in the hope that it will be useful, but WITHOUT + ANY WARRANTY; without even the implied warranty of MERCHANTABILITY + or FITNESS FOR A PARTICULAR PURPOSE. See the GNU General Public + License for more details. + You should have received a copy of the GNU General Public License along with this program; if not, write to the Free Software Foundation, Inc., - 59 Temple Place - Suite 330, Boston, MA 02111-1307, USA. */ + 51 Franklin Street - Fifth Floor, Boston, MA 02110-1301, USA. */ #include "sysdep.h" #include -#include #include "ansidecl.h" #include "libiberty.h" +#include "safe-ctype.h" #include "bfd.h" #include "symcat.h" #include "opcode/cgen.h" #include "opintl.h" -/* Operand parsing callback. */ -const char * (*cgen_parse_operand_fn) - PARAMS ((enum cgen_parse_operand_type, const char **, int, int, - enum cgen_parse_operand_result *, bfd_vma *)); - -/* This is not published as part of the public interface so we don't - declare this in cgen.h. */ -extern CGEN_OPCODE_TABLE *cgen_current_opcode_table; +static CGEN_INSN_LIST * hash_insn_array (CGEN_CPU_DESC, const CGEN_INSN *, int, int, CGEN_INSN_LIST **, CGEN_INSN_LIST *); +static CGEN_INSN_LIST * hash_insn_list (CGEN_CPU_DESC, const CGEN_INSN_LIST *, CGEN_INSN_LIST **, CGEN_INSN_LIST *); +static void build_asm_hash_table (CGEN_CPU_DESC); -/* Assembler instruction hash table. */ -static CGEN_INSN_LIST **asm_hash_table; -static CGEN_INSN_LIST *asm_hash_table_entries; - -/* Called once at startup and whenever machine/endian change. */ +/* Set the cgen_parse_operand_fn callback. */ void -cgen_asm_init () +cgen_set_parse_operand_fn (CGEN_CPU_DESC cd, cgen_parse_operand_fn fn) { - if (asm_hash_table) - { - free (asm_hash_table); - free (asm_hash_table_entries); - asm_hash_table = NULL; - asm_hash_table_entries = NULL; - } + cd->parse_operand_fn = fn; } /* Called whenever starting to parse an insn. */ void -cgen_init_parse_operand () +cgen_init_parse_operand (CGEN_CPU_DESC cd) { /* This tells the callback to re-initialize. */ - (void) (*cgen_parse_operand_fn) (CGEN_PARSE_OPERAND_INIT, NULL, 0, 0, - NULL, NULL); + (void) (* cd->parse_operand_fn) + (cd, CGEN_PARSE_OPERAND_INIT, NULL, 0, 0, NULL, NULL); } /* Subroutine of build_asm_hash_table to add INSNS to the hash table. COUNT is the number of elements in INSNS. - ENTSIZE is sizeof (CGEN_INSN) for the target. - This is a bit tricky as the size of the attribute member of CGEN_INSN - is variable among architectures. This code could be moved to - cgen-asm.in, but I prefer to keep it here for now. - OTABLE is the opcode table. + ENTSIZE is sizeof (CGEN_IBASE) for the target. + ??? No longer used but leave in for now. HTABLE points to the hash table. HENTBUF is a pointer to sufficiently large buffer of hash entries. The result is a pointer to the next entry to use. @@ -81,25 +63,23 @@ cgen_init_parse_operand () list and we want earlier ones to be prefered. */ static CGEN_INSN_LIST * -hash_insn_array (insns, count, entsize, otable, htable, hentbuf) - const CGEN_INSN * insns; - int count; - int entsize; - const CGEN_OPCODE_TABLE * otable; - CGEN_INSN_LIST ** htable; - CGEN_INSN_LIST * hentbuf; +hash_insn_array (CGEN_CPU_DESC cd, + const CGEN_INSN *insns, + int count, + int entsize ATTRIBUTE_UNUSED, + CGEN_INSN_LIST **htable, + CGEN_INSN_LIST *hentbuf) { - const CGEN_INSN * insn; + int i; - for (insn = (CGEN_INSN *) ((char *) insns + entsize * (count - 1)); - insn >= insns; - insn = (CGEN_INSN *) ((char *) insn - entsize), ++ hentbuf) + for (i = count - 1; i >= 0; --i, ++hentbuf) { unsigned int hash; + const CGEN_INSN *insn = &insns[i]; - if (! (*otable->asm_hash_p) (insn)) + if (! (* cd->asm_hash_p) (insn)) continue; - hash = (*otable->asm_hash) (CGEN_INSN_MNEMONIC (insn)); + hash = (* cd->asm_hash) (CGEN_INSN_MNEMONIC (insn)); hentbuf->next = htable[hash]; hentbuf->insn = insn; htable[hash] = hentbuf; @@ -113,24 +93,23 @@ hash_insn_array (insns, count, entsize, otable, htable, hentbuf) in a list. */ static CGEN_INSN_LIST * -hash_insn_list (insns, otable, htable, hentbuf) - const CGEN_INSN_LIST * insns; - const CGEN_OPCODE_TABLE * otable; - CGEN_INSN_LIST ** htable; - CGEN_INSN_LIST * hentbuf; +hash_insn_list (CGEN_CPU_DESC cd, + const CGEN_INSN_LIST *insns, + CGEN_INSN_LIST **htable, + CGEN_INSN_LIST *hentbuf) { - const CGEN_INSN_LIST * ilist; + const CGEN_INSN_LIST *ilist; for (ilist = insns; ilist != NULL; ilist = ilist->next, ++ hentbuf) { unsigned int hash; - if (! (*otable->asm_hash_p) (ilist->insn)) + if (! (* cd->asm_hash_p) (ilist->insn)) continue; - hash = (*otable->asm_hash) (CGEN_INSN_MNEMONIC (ilist->insn)); + hash = (* cd->asm_hash) (CGEN_INSN_MNEMONIC (ilist->insn)); hentbuf->next = htable[hash]; hentbuf->insn = ilist->insn; - asm_hash_table[hash] = hentbuf; + htable[hash] = hentbuf; } return hentbuf; @@ -139,14 +118,15 @@ hash_insn_list (insns, otable, htable, hentbuf) /* Build the assembler instruction hash table. */ static void -build_asm_hash_table () +build_asm_hash_table (CGEN_CPU_DESC cd) { - int count = cgen_insn_count () + cgen_macro_insn_count (); - CGEN_OPCODE_TABLE *opcode_table = cgen_current_opcode_table; - CGEN_INSN_TABLE *insn_table = opcode_table->insn_table; - CGEN_INSN_TABLE *macro_insn_table = opcode_table->macro_insn_table; - unsigned int hash_size = opcode_table->asm_hash_table_size; + int count = cgen_insn_count (cd) + cgen_macro_insn_count (cd); + CGEN_INSN_TABLE *insn_table = &cd->insn_table; + CGEN_INSN_TABLE *macro_insn_table = &cd->macro_insn_table; + unsigned int hash_size = cd->asm_hash_size; CGEN_INSN_LIST *hash_entry_buf; + CGEN_INSN_LIST **asm_hash_table; + CGEN_INSN_LIST *asm_hash_table_entries; /* The space allocated for the hash table consists of two parts: the hash table and the hash lists. */ @@ -160,48 +140,48 @@ build_asm_hash_table () /* Add compiled in insns. Don't include the first one as it is a reserved entry. */ /* ??? It was the end of all hash chains, and also the special - "illegal insn" marker. May be able to do it differently now. */ + "invalid insn" marker. May be able to do it differently now. */ - hash_entry_buf = hash_insn_array ((CGEN_INSN *) ((char *) insn_table->init_entries - + insn_table->entry_size), + hash_entry_buf = hash_insn_array (cd, + insn_table->init_entries + 1, insn_table->num_init_entries - 1, insn_table->entry_size, - opcode_table, asm_hash_table, hash_entry_buf); + asm_hash_table, hash_entry_buf); /* Add compiled in macro-insns. */ - hash_entry_buf = hash_insn_array (macro_insn_table->init_entries, + hash_entry_buf = hash_insn_array (cd, macro_insn_table->init_entries, macro_insn_table->num_init_entries, macro_insn_table->entry_size, - opcode_table, asm_hash_table, hash_entry_buf); + asm_hash_table, hash_entry_buf); /* Add runtime added insns. Later added insns will be prefered over earlier ones. */ - hash_entry_buf = hash_insn_list (insn_table->new_entries, opcode_table, + hash_entry_buf = hash_insn_list (cd, insn_table->new_entries, asm_hash_table, hash_entry_buf); /* Add runtime added macro-insns. */ - hash_insn_list (macro_insn_table->new_entries, - opcode_table, asm_hash_table, hash_entry_buf); + hash_insn_list (cd, macro_insn_table->new_entries, + asm_hash_table, hash_entry_buf); + + cd->asm_hash_table = asm_hash_table; + cd->asm_hash_table_entries = asm_hash_table_entries; } -/* Return the first entry in the hash list for INSN. - ??? Of course it would be better to pass in a pointer to the - opcode data structure, rather than reference a global. Later. */ +/* Return the first entry in the hash list for INSN. */ CGEN_INSN_LIST * -cgen_asm_lookup_insn (insn) - const char *insn; +cgen_asm_lookup_insn (CGEN_CPU_DESC cd, const char *insn) { unsigned int hash; - if (asm_hash_table == NULL) - build_asm_hash_table (); + if (cd->asm_hash_table == NULL) + build_asm_hash_table (cd); - hash = (*cgen_current_opcode_table->asm_hash) (insn); - return asm_hash_table[hash]; + hash = (* cd->asm_hash) (insn); + return cd->asm_hash_table[hash]; } /* Keyword parser. @@ -213,33 +193,45 @@ cgen_asm_lookup_insn (insn) recording something in the keyword table]. */ const char * -cgen_parse_keyword (strp, keyword_table, valuep) - const char **strp; - CGEN_KEYWORD *keyword_table; - long *valuep; +cgen_parse_keyword (CGEN_CPU_DESC cd ATTRIBUTE_UNUSED, + const char **strp, + CGEN_KEYWORD *keyword_table, + long *valuep) { const CGEN_KEYWORD_ENTRY *ke; char buf[256]; const char *p,*start; + if (keyword_table->name_hash_table == NULL) + (void) cgen_keyword_search_init (keyword_table, NULL); + p = start = *strp; - /* Allow any first character. - Note that this allows recognizing ",a" for the annul flag in sparc - even though "," is subsequently not a valid keyword char. */ + /* Allow any first character. This is to make life easier for + the fairly common case of suffixes, eg. 'ld.b.w', where the first + character of the suffix ('.') is special. */ if (*p) ++p; - /* Now allow letters, digits, and _. */ + /* Allow letters, digits, and any special characters. */ while (((p - start) < (int) sizeof (buf)) - && (isalnum ((unsigned char) *p) || *p == '_')) + && *p + && (ISALNUM (*p) + || *p == '_' + || strchr (keyword_table->nonalpha_chars, *p))) ++p; if (p - start >= (int) sizeof (buf)) - return _("unrecognized keyword/register name"); - - memcpy (buf, start, p - start); - buf[p - start] = 0; + { + /* All non-empty CGEN keywords can fit into BUF. The only thing + we can match here is the empty keyword. */ + buf[0] = 0; + } + else + { + memcpy (buf, start, p - start); + buf[p - start] = 0; + } ke = cgen_keyword_lookup_name (keyword_table, buf); @@ -255,42 +247,64 @@ cgen_parse_keyword (strp, keyword_table, valuep) return "unrecognized keyword/register name"; } -/* Signed integer parser. */ +/* Parse a small signed integer parser. + ??? VALUEP is not a bfd_vma * on purpose, though this is confusing. + Note that if the caller expects a bfd_vma result, it should call + cgen_parse_address. */ const char * -cgen_parse_signed_integer (strp, opindex, valuep) - const char **strp; - int opindex; - long *valuep; +cgen_parse_signed_integer (CGEN_CPU_DESC cd, + const char **strp, + int opindex, + long *valuep) { bfd_vma value; enum cgen_parse_operand_result result; const char *errmsg; - errmsg = (*cgen_parse_operand_fn) (CGEN_PARSE_OPERAND_INTEGER, strp, - opindex, BFD_RELOC_NONE, - &result, &value); + errmsg = (* cd->parse_operand_fn) + (cd, CGEN_PARSE_OPERAND_INTEGER, strp, opindex, BFD_RELOC_NONE, + &result, &value); /* FIXME: Examine `result'. */ if (!errmsg) - *valuep = value; + { + /* Handle the case where a hex value is parsed on a 64-bit host. + A value like 0xffffe000 is clearly intended to be a negative + 16-bit value, but on a 64-bit host it will be parsed by gas + as 0x00000000ffffe000. + + The shifts below are designed not to produce compile time + warnings on a 32-bit host. */ + if (sizeof (value) > 4 + && result == CGEN_PARSE_OPERAND_RESULT_NUMBER + && value > 0 + && (value & 0x80000000) + && ((value >> 31) == 1)) + value |= ((bfd_vma) -1) << 31; + + *valuep = value; + } return errmsg; } -/* Unsigned integer parser. */ +/* Parse a small unsigned integer parser. + ??? VALUEP is not a bfd_vma * on purpose, though this is confusing. + Note that if the caller expects a bfd_vma result, it should call + cgen_parse_address. */ const char * -cgen_parse_unsigned_integer (strp, opindex, valuep) - const char **strp; - int opindex; - unsigned long *valuep; +cgen_parse_unsigned_integer (CGEN_CPU_DESC cd, + const char **strp, + int opindex, + unsigned long *valuep) { bfd_vma value; enum cgen_parse_operand_result result; const char *errmsg; - errmsg = (*cgen_parse_operand_fn) (CGEN_PARSE_OPERAND_INTEGER, strp, - opindex, BFD_RELOC_NONE, - &result, &value); + errmsg = (* cd->parse_operand_fn) + (cd, CGEN_PARSE_OPERAND_INTEGER, strp, opindex, BFD_RELOC_NONE, + &result, &value); /* FIXME: Examine `result'. */ if (!errmsg) *valuep = value; @@ -300,20 +314,20 @@ cgen_parse_unsigned_integer (strp, opindex, valuep) /* Address parser. */ const char * -cgen_parse_address (strp, opindex, opinfo, resultp, valuep) - const char **strp; - int opindex; - int opinfo; - enum cgen_parse_operand_result *resultp; - long *valuep; +cgen_parse_address (CGEN_CPU_DESC cd, + const char **strp, + int opindex, + int opinfo, + enum cgen_parse_operand_result *resultp, + bfd_vma *valuep) { bfd_vma value; enum cgen_parse_operand_result result_type; const char *errmsg; - errmsg = (*cgen_parse_operand_fn) (CGEN_PARSE_OPERAND_ADDRESS, strp, - opindex, opinfo, - &result_type, &value); + errmsg = (* cd->parse_operand_fn) + (cd, CGEN_PARSE_OPERAND_ADDRESS, strp, opindex, opinfo, + &result_type, &value); /* FIXME: Examine `result'. */ if (!errmsg) { @@ -327,8 +341,7 @@ cgen_parse_address (strp, opindex, opinfo, resultp, valuep) /* Signed integer validation routine. */ const char * -cgen_validate_signed_integer (value, min, max) - long value, min, max; +cgen_validate_signed_integer (long value, long min, long max) { if (value < min || value > max) { @@ -348,13 +361,15 @@ cgen_validate_signed_integer (value, min, max) cases where min != 0 (and max > LONG_MAX). */ const char * -cgen_validate_unsigned_integer (value, min, max) - unsigned long value, min, max; +cgen_validate_unsigned_integer (unsigned long value, + unsigned long min, + unsigned long max) { if (value < min || value > max) { static char buf[100]; + /* xgettext:c-format */ sprintf (buf, _("operand out of range (%lu not between %lu and %lu)"), value, min, max); return buf;