* Re: [patch] gas/config/tc-h8300.c
[not found] <200006151611.JAA24092@cygnus.com>
@ 2000-06-21 14:31 ` Jeffrey A Law
0 siblings, 0 replies; 6+ messages in thread
From: Jeffrey A Law @ 2000-06-21 14:31 UTC (permalink / raw)
To: Kazu Hirata; +Cc: binutils
In message <200006151611.JAA24092@cygnus.com>you write:
> Hi,
>
> Attached is a patch for gas/config/tc-h8300.c. It makes two functions
> static, which are not used anywhere else.
>
> Thanks,
>
> Kazu Hirata
>
> ===File ~/gnu/binutils/ChangeLog-tc-h8300===================
> 2000-06-15 Kazu Hirata <kazu@hxi.com>
>
> * config/tc-h8300.c (parse_reg): Make the function static.
> (parse_exp): Likewise.
Thanks. Installed.
jeff
^ permalink raw reply [flat|nested] 6+ messages in thread
* [patch] gas/config/tc-h8300.c
@ 2000-06-22 9:47 Kazu Hirata
2000-06-22 22:44 ` Alan Modra
0 siblings, 1 reply; 6+ messages in thread
From: Kazu Hirata @ 2000-06-22 9:47 UTC (permalink / raw)
To: 'binutils@sourceware.cygnus.com'
[-- Attachment #1: Type: text/plain, Size: 96 bytes --]
Hi,
Attached is a patch for gas/config/tc-h8300.c. It fixes formatting.
Thanks,
Kazu Hirata
[-- Attachment #2: ChangeLog-tc-h8300 --]
[-- Type: text/plain, Size: 80 bytes --]
2000-06-22 Kazu Hirata <kazu@hxi.com>
* config/tc-h8300.c: Fix formatting.
[-- Attachment #3: tc-h8300.patch --]
[-- Type: text/x-diff, Size: 15102 bytes --]
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 */
^ permalink raw reply [flat|nested] 6+ messages in thread
* [patch] gas/config/tc-h8300.c
@ 2000-06-15 9:11 Kazu Hirata
0 siblings, 0 replies; 6+ messages in thread
From: Kazu Hirata @ 2000-06-15 9:11 UTC (permalink / raw)
To: binutils
Hi,
Attached is a patch for gas/config/tc-h8300.c. It makes two functions
static, which are not used anywhere else.
Thanks,
Kazu Hirata
===File ~/gnu/binutils/ChangeLog-tc-h8300===================
2000-06-15 Kazu Hirata <kazu@hxi.com>
* config/tc-h8300.c (parse_reg): Make the function static.
(parse_exp): Likewise.
============================================================
===File ~/gnu/binutils/tc-h8300.patch=======================
Index: tc-h8300.c
===================================================================
RCS file: /cvs/src/src/gas/config/tc-h8300.c,v
retrieving revision 1.4
diff -u -r1.4 tc-h8300.c
--- tc-h8300.c 2000/06/09 00:00:02 1.4
+++ tc-h8300.c 2000/06/15 16:03:17
@@ -200,8 +200,8 @@
ccr
*/
-/* try and parse a reg name, returns number of chars consumed */
-int
+/* Try and parse a reg name, returns number of chars consumed. */
+static int
parse_reg (src, mode, reg, direction)
char *src;
op_type *mode;
@@ -290,7 +290,7 @@
return 0;
}
-char *
+static char *
parse_exp (s, op)
char *s;
expressionS * op;
============================================================
^ permalink raw reply [flat|nested] 6+ messages in thread
* [patch] gas/config/tc-h8300.c
@ 2000-05-07 8:35 Kazu Hirata
2000-05-14 4:50 ` Alan Modra
0 siblings, 1 reply; 6+ messages in thread
From: Kazu Hirata @ 2000-05-07 8:35 UTC (permalink / raw)
To: 'binutils@sourceware.cygnus.com'
[-- Attachment #1: Type: text/plain, Size: 335 bytes --]
Hi,
Attached is a patch for Hitachi H8 series. With this patch, gas outputs a
reloc for a mov instruction so that ld will consider relaxing the
instruction. The linker shuold not be confused as a result of this patch.
The implementation is as in:
http://sourceware.cygnus.com/ml/binutils/2000-03/msg00266.html
Thanks,
Kazu Hirata
[-- Attachment #2: ChangeLog-relax --]
[-- Type: text/plain, Size: 109 bytes --]
2000-05-07 Kazu Hirata <kazu@hxi.com>
* config/tc-h8300.c (do_a_fix_imm): Output a reloc for relaxation.
[-- Attachment #3: h8300-hms-binutils-relax.patch --]
[-- Type: text/x-diff, Size: 579 bytes --]
diff -ruN binutils-2.9.5.0.41-old/gas/config/tc-h8300.c binutils-2.9.5.0.41/gas/config/tc-h8300.c
--- binutils-2.9.5.0.41-old/gas/config/tc-h8300.c Sun Mar 19 14:55:07 2000
+++ binutils-2.9.5.0.41/gas/config/tc-h8300.c Sun May 7 10:18:48 2000
@@ -886,6 +886,11 @@
bytes[1] = operand->exp.X_add_number >> 16;
bytes[2] = operand->exp.X_add_number >> 8;
bytes[3] = operand->exp.X_add_number >> 0;
+ if (relaxmode)
+ {
+ idx = (relaxmode == 2) ? R_MOV24B1 : R_MOVL1;
+ fix_new_exp (frag_now, offset, 4, &operand->exp, 0, idx);
+ }
break;
}
^ permalink raw reply [flat|nested] 6+ messages in thread
end of thread, other threads:[~2000-06-22 22:44 UTC | newest]
Thread overview: 6+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
[not found] <200006151611.JAA24092@cygnus.com>
2000-06-21 14:31 ` [patch] gas/config/tc-h8300.c Jeffrey A Law
2000-06-22 9:47 Kazu Hirata
2000-06-22 22:44 ` Alan Modra
-- strict thread matches above, loose matches on Subject: below --
2000-06-15 9:11 Kazu Hirata
2000-05-07 8:35 Kazu Hirata
2000-05-14 4:50 ` Alan Modra
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for read-only IMAP folder(s) and NNTP newsgroup(s).