Index: gas/config/tc-h8300.c =================================================================== RCS file: /cvs/src/src/gas/config/tc-h8300.c,v retrieving revision 1.5 diff -u -r1.5 tc-h8300.c --- tc-h8300.c 2000/06/21 21:31:24 1.5 +++ tc-h8300.c 2000/06/22 16:24:49 @@ -19,7 +19,6 @@ Software Foundation, 59 Temple Place - Suite 330, Boston, MA 02111-1307, USA. */ - /* Written By Steve Chamberlain sac@cygnus.com @@ -54,7 +53,6 @@ #define DSYMMODE (Hmode ? L_24 : L_16) int bsize = L_8; /* default branch displacement */ - void h8300hmode () { @@ -68,6 +66,7 @@ Smode = 1; Hmode = 1; } + void sbranch (size) int size; @@ -75,14 +74,14 @@ bsize = size; } -static void pint () +static void +pint () { cons (Hmode ? 4 : 2); } const pseudo_typeS md_pseudo_table[] = { - {"h8300h", h8300hmode, 0}, {"h8300s", h8300smode, 0}, {"sbranch", sbranch, L_8}, @@ -110,13 +109,10 @@ const char FLT_CHARS[] = "rRsSfFdDxXpP"; static struct hash_control *opcode_hash_control; /* Opcode mnemonics */ - -/* - This function is called once, at assembler startup time. This should - set up all the tables, etc that the MD part of the assembler needs - */ - +/* This function is called once, at assembler startup time. This + should set up all the tables, etc that the MD part of the assembler + needs. */ void md_begin () { @@ -130,8 +126,7 @@ for (opcode = h8_opcodes; opcode->name; opcode++) { /* Strip off any . part when inserting the opcode and only enter - unique codes into the hash table - */ + unique codes into the hash table. */ char *src = opcode->name; unsigned int len = strlen (src); char *dst = malloc (len + 1); @@ -156,13 +151,13 @@ idx++; } opcode->idx = idx; - - /* Find the number of operands */ + /* Find the number of operands. */ opcode->noperands = 0; while (opcode->args.nib[opcode->noperands] != E) opcode->noperands++; - /* Find the length of the opcode in bytes */ + + /* Find the length of the opcode in bytes. */ opcode->length = 0; while (opcode->data.nib[opcode->length * 2] != E) opcode->length++; @@ -171,17 +166,16 @@ linkrelax = 1; } - struct h8_exp { char *e_beg; char *e_end; expressionS e_exp; }; + int dispreg; int opsize; /* Set when a register size is seen */ - struct h8_op { op_type mode; @@ -212,10 +206,10 @@ char *end; int len; - /* Cribbed from get_symbol_end(). */ + /* Cribbed from get_symbol_end (). */ if (!is_name_beginner (*src) || *src == '\001') return 0; - end = src+1; + end = src + 1; while (is_part_of_name (*end) || *end == '\001') end++; len = end - src; @@ -293,7 +287,7 @@ static char * parse_exp (s, op) char *s; - expressionS * op; + expressionS *op; { char *save = input_line_pointer; char *new; @@ -321,7 +315,7 @@ { ptr++; /* ff fill any 8 bit quantity */ - /* exp->X_add_number -= 0x100;*/ + /* exp->X_add_number -= 0x100; */ *mode |= L_8; } else @@ -373,7 +367,7 @@ if (!mode) { - /* Choose a default mode */ + /* Choose a default mode. */ if (op->exp.X_add_number < -32768 || op->exp.X_add_number > 32767) { @@ -393,7 +387,6 @@ } - static void get_operand (ptr, op, dst, direction) char **ptr; @@ -410,8 +403,8 @@ /* Gross. Gross. ldm and stm have a format not easily handled by get_operand. We deal with it explicitly here. */ - if (src[0] == 'e' && src[1] == 'r' && isdigit(src[2]) - && src[3] == '-' && src[4] == 'e' && src[5] == 'r' && isdigit(src[6])) + if (src[0] == 'e' && src[1] == 'r' && isdigit (src[2]) + && src[3] == '-' && src[4] == 'e' && src[5] == 'r' && isdigit (src[6])) { int low, high; @@ -462,29 +455,24 @@ op->mode = MEMIND; return; - } - if (*src == '-') { src++; len = parse_reg (src, &mode, &num, direction); if (len == 0) { - /* Oops, not a reg after all, must be ordinary exp */ + /* Oops, not a reg after all, must be ordinary exp. */ src--; - /* must be a symbol */ + /* Must be a symbol. */ op->mode = ABS | PSIZE | direction; *ptr = skip_colonthing (parse_exp (src, &op->exp), &op->exp, &op->mode); return; - - } - if ((mode & SIZE) != PSIZE) as_bad (_("Wrong size pointer register for architecture.")); op->mode = RDDEC; @@ -494,12 +482,11 @@ } if (*src == '(') { - /* Disp */ + /* Disp. */ src++; - /* Start off assuming a 16 bit offset */ + /* Start off assuming a 16 bit offset. */ - src = parse_exp (src, &op->exp); src = colonmod24 (op, src); @@ -577,7 +564,6 @@ } } - if (*src == '#') { src++; @@ -624,9 +610,7 @@ } } - -static -char * +static char * get_operands (noperands, op_end, operand) unsigned int noperands; char *op_end; @@ -653,8 +637,8 @@ { operand[1].mode = 0; } - break; + case 2: ptr++; get_operand (&ptr, operand + 0, 0, SRC); @@ -667,16 +651,13 @@ abort (); } - return ptr; } /* Passed a pointer to a list of opcodes which use different addressing modes, return the opcode which matches the opcodes - provided - */ -static -struct h8_opcode * + provided. */ +static struct h8_opcode * get_specific (opcode, operands, size) struct h8_opcode *opcode; struct h8_op *operands; @@ -732,7 +713,7 @@ opsize = op & SIZE; - /* The size of the reg is v important */ + /* The size of the reg is v important. */ if ((op & SIZE) != (x & SIZE)) found = 0; } @@ -740,7 +721,7 @@ { operands[i].mode &= ~ABS; operands[i].mode |= ABSJMP; - /* But it may not be 24 bits long */ + /* But it may not be 24 bits long. */ if (!Hmode) { operands[i].mode &= ~SIZE; @@ -749,11 +730,11 @@ } else if ((op & (KBIT | DBIT)) && (x & IMM)) { - /* This is ok if the immediate value is sensible */ + /* This is ok if the immediate value is sensible. */ } else if (op & PCREL) { - /* The size of the displacement is important */ + /* The size of the displacement is important. */ if ((op & SIZE) != (x & SIZE)) found = 0; } @@ -767,10 +748,10 @@ x |= L_32; } /* Promote an L8 to L_16 if it makes us match. */ - if (op & ABS && op & L_8 && op & DISP) + if (op & ABS && op & L_8 && op & DISP) { if (x & L_16) - found= 1; + found = 1; } else if ((x & SIZE) != 0 && ((op & SIZE) != (x & SIZE))) @@ -783,7 +764,7 @@ else if ((op & MODE) != (x & MODE)) { found = 0; - } + } } } } @@ -802,10 +783,10 @@ if (operand->exp.X_add_symbol == 0 && operand->exp.X_op_symbol == 0) { - - /* No symbol involved, let's look at offset, it's dangerous if any of - the high bits are not 0 or ff's, find out by oring or anding with - the width and seeing if the answer is 0 or all fs*/ + /* No symbol involved, let's look at offset, it's dangerous if + any of the high bits are not 0 or ff's, find out by oring or + anding with the width and seeing if the answer is 0 or all + fs. */ if ((operand->exp.X_add_number & ~width) != 0 && (operand->exp.X_add_number | width) != (~0)) @@ -817,14 +798,13 @@ fit a 16 bit address truncated into an 8 bit address of something like bset. */ } - else + else { as_warn (_("operand %s0x%lx out of range."), string, (unsigned long) operand->exp.X_add_number); } } } - } /* RELAXMODE has one of 3 values: @@ -847,7 +827,6 @@ int size; int where; - char *t = operand->mode & IMM ? "#" : "@"; if (operand->exp.X_add_symbol == 0) @@ -880,7 +859,7 @@ break; case L_32: - /* This should be done with bfd */ + /* This should be done with bfd. */ bytes[0] = operand->exp.X_add_number >> 24; bytes[1] = operand->exp.X_add_number >> 16; bytes[2] = operand->exp.X_add_number >> 8; @@ -892,13 +871,11 @@ } break; } - } else { switch (operand->mode & SIZE) { - case L_24: case L_32: size = 4; @@ -911,7 +888,7 @@ idx = R_RELLONG; break; default: - as_bad(_("Can't work out size of operand.\n")); + as_bad (_("Can't work out size of operand.\n")); case L_16: size = 2; where = 0; @@ -937,11 +914,9 @@ 0, idx); } - } -/* Now we know what sort of opcodes it is, lets build the bytes - - */ +/* Now we know what sort of opcodes it is, lets build the bytes. */ static void build_bytes (this_try, operand) struct h8_opcode *this_try; @@ -977,7 +952,6 @@ } else { - if (c & (REG | IND | INC | DEC)) { nib = operand[d].reg; @@ -986,7 +960,7 @@ { nib = dispreg; } - else if (c & ABS ) + else if (c & ABS) { operand[d].mode = c; absat = nibble_count / 2; @@ -1036,7 +1010,7 @@ as_bad (_("Need #1 or #2 here")); break; } - /* stop it making a fix */ + /* Stop it making a fix. */ operand[0].mode = 0; } @@ -1083,7 +1057,7 @@ if (strcmp (this_try->name, "mov.b") == 0) movb = 1; - /* output any fixes */ + /* Output any fixes. */ for (i = 0; i < 2; i++) { int x = operand[i].mode; @@ -1126,7 +1100,6 @@ } else if (x & MEMIND) { - check_operand (operand + i, 0xff, "@@"); fix_new_exp (frag_now, output - frag_now->fr_literal + 1, @@ -1137,7 +1110,7 @@ } else if (x & ABSJMP) { - /* This jmp may be a jump or a branch */ + /* This jmp may be a jump or a branch. */ check_operand (operand + i, Hmode ? 0xffffff : 0xffff, "@"); if (operand[i].exp.X_add_number & 1) @@ -1146,7 +1119,7 @@ (unsigned long) operand->exp.X_add_number); } if (!Hmode) - operand[i].exp.X_add_number = + operand[i].exp.X_add_number = ((operand[i].exp.X_add_number & 0xffff) ^ 0x8000) - 0x8000; fix_new_exp (frag_now, output - frag_now->fr_literal, @@ -1156,27 +1129,23 @@ R_JMPL1); } } - } -/* - try and give an intelligent error message for common and simple to - detect errors - */ - +/* Try and give an intelligent error message for common and simple to + detect errors. */ static void clever_message (opcode, operand) struct h8_opcode *opcode; struct h8_op *operand; { - /* Find out if there was more than one possible opccode */ + /* Find out if there was more than one possible opccode. */ if ((opcode + 1)->idx != opcode->idx) { unsigned int argn; /* Only one opcode of this flavour, try and guess which operand - didn't match */ + didn't match. */ for (argn = 0; argn < opcode->noperands; argn++) { switch (opcode->args.nib[argn]) @@ -1191,7 +1160,6 @@ break; case RS8: - if (operand[argn].mode != RS8) { as_bad (_("source operand must be 8 bit register")); @@ -1214,7 +1182,6 @@ } break; - case ABS16SRC: if (operand[argn].mode != ABS16SRC) { @@ -1228,14 +1195,10 @@ } as_bad (_("invalid operands")); } - -/* This is the guts of the machine-dependent assembler. STR points to a - machine dependent instruction. This funciton is supposed to emit - the frags/bytes it assembles to. - */ - - +/* This is the guts of the machine-dependent assembler. STR points to + a machine dependent instruction. This funciton is supposed to emit + the frags/bytes it assembles to. */ void md_assemble (str) char *str; @@ -1250,11 +1213,11 @@ char c; int size; - /* Drop leading whitespace */ + /* Drop leading whitespace. */ while (*str == ' ') str++; - /* find the op code end */ + /* Find the op code end. */ for (op_start = op_end = str; *op_end != 0 && *op_end != ' '; op_end++) @@ -1268,8 +1231,6 @@ } } - ; - if (op_end == op_start) { as_bad (_("can't find opcode ")); @@ -1316,7 +1277,7 @@ if (opcode == 0) { - /* Couldn't find an opcode which matched the operands */ + /* Couldn't find an opcode which matched the operands. */ char *where = frag_more (2); where[0] = 0x0; @@ -1334,12 +1295,11 @@ } build_bytes (opcode, operand); - } void tc_crawl_symbol_chain (headers) - object_headers * headers ATTRIBUTE_UNUSED; + object_headers *headers ATTRIBUTE_UNUSED; { printf (_("call to tc_crawl_symbol_chain \n")); } @@ -1353,7 +1313,7 @@ void tc_headers_hook (headers) - object_headers * headers ATTRIBUTE_UNUSED; + object_headers *headers ATTRIBUTE_UNUSED; { printf (_("call to tc_headers_hook \n")); } @@ -1362,10 +1322,10 @@ /* Equal to MAX_PRECISION in atof-ieee.c */ #define MAX_LITTLENUMS 6 -/* Turn a string in input_line_pointer into a floating point constant of type - type, and store the appropriate bytes in *litP. The number of LITTLENUMS - emitted is stored in *sizeP . An error message is returned, or NULL on OK. - */ +/* Turn a string in input_line_pointer into a floating point constant + of type type, and store the appropriate bytes in *litP. The number + of LITTLENUMS emitted is stored in *sizeP . An error message is + returned, or NULL on OK. */ char * md_atof (type, litP, sizeP) char type; @@ -1425,8 +1385,9 @@ struct option md_longopts[] = { {NULL, no_argument, NULL, 0} }; -size_t md_longopts_size = sizeof(md_longopts); +size_t md_longopts_size = sizeof (md_longopts); + int md_parse_option (c, arg) int c ATTRIBUTE_UNUSED; @@ -1463,8 +1424,8 @@ segT seg; valueT size; { - return ((size + (1 << section_alignment[(int) seg]) - 1) & (-1 << section_alignment[(int) seg])); - + return ((size + (1 << section_alignment[(int) seg]) - 1) + & (-1 << section_alignment[(int) seg])); } void @@ -1503,8 +1464,7 @@ abort (); } -/* Put number into target byte order */ - +/* Put number into target byte order. */ void md_number_to_chars (ptr, use, nbytes) char *ptr; @@ -1513,6 +1473,7 @@ { number_to_chars_bigendian (ptr, use, nbytes); } + long md_pcrel_from (fixP) fixS *fixP ATTRIBUTE_UNUSED; @@ -1520,7 +1481,6 @@ abort (); } - void tc_reloc_mangle (fix_ptr, intr, base) fixS *fix_ptr; @@ -1533,7 +1493,7 @@ symbol_ptr = fix_ptr->fx_addsy; /* If this relocation is attached to a symbol then it's ok - to output it */ + to output it. */ if (fix_ptr->fx_r_type == TC_CONS_RELOC) { /* cons likes to create reloc32's whatever the size of the reloc.. @@ -1551,9 +1511,7 @@ break; default: abort (); - } - } else { @@ -1584,8 +1542,4 @@ } else intr->r_symndx = -1; - - } - -/* end of tc-h8300.c */