* config/tc-mips.c (macro_build_lui): Fix comment formatting.

(md_apply_fix): Likewise. Unify handling of BFD_RELOC_RVA,
	BFD_RELOC_32 and BFD_RELOC_16.
	(s_align, s_cpload, s_cplocal, s_cprestore, s_mips_stab,
	md_convert_frag, md_obj_end): Fix comment formatting.
This commit is contained in:
Thiemo Seufer
2006-08-01 05:49:02 +00:00
parent ae55530d3d
commit 54f4ddb3c6
2 changed files with 33 additions and 39 deletions

View File

@ -1,3 +1,11 @@
2006-08-01 Thiemo Seufer <ths@mips.com>
* config/tc-mips.c (macro_build_lui): Fix comment formatting.
(md_apply_fix): Likewise. Unify handling of BFD_RELOC_RVA,
BFD_RELOC_32 and BFD_RELOC_16.
(s_align, s_cpload, s_cplocal, s_cprestore, s_mips_stab,
md_convert_frag, md_obj_end): Fix comment formatting.
2006-07-31 Thiemo Seufer <ths@mips.com> 2006-07-31 Thiemo Seufer <ths@mips.com>
* config/tc-mips.c (md_apply_fix, tc_gen_reloc): Remove special * config/tc-mips.c (md_apply_fix, tc_gen_reloc): Remove special

View File

@ -3719,7 +3719,7 @@ macro_build_lui (expressionS *ep, int regnum)
if (high_expr.X_op == O_constant) if (high_expr.X_op == O_constant)
{ {
/* we can compute the instruction now without a relocation entry */ /* We can compute the instruction now without a relocation entry. */
high_expr.X_add_number = ((high_expr.X_add_number + 0x8000) high_expr.X_add_number = ((high_expr.X_add_number + 0x8000)
>> 16) & 0xffff; >> 16) & 0xffff;
*r = BFD_RELOC_UNUSED; *r = BFD_RELOC_UNUSED;
@ -11788,7 +11788,7 @@ md_apply_fix (fixS *fixP, valueT *valP, segT seg ATTRIBUTE_UNUSED)
case BFD_RELOC_MIPS16_HI16: case BFD_RELOC_MIPS16_HI16:
case BFD_RELOC_MIPS16_HI16_S: case BFD_RELOC_MIPS16_HI16_S:
case BFD_RELOC_MIPS16_JMP: case BFD_RELOC_MIPS16_JMP:
/* Nothing needed to do. The value comes from the reloc entry */ /* Nothing needed to do. The value comes from the reloc entry. */
break; break;
case BFD_RELOC_64: case BFD_RELOC_64:
@ -11816,18 +11816,12 @@ md_apply_fix (fixS *fixP, valueT *valP, segT seg ATTRIBUTE_UNUSED)
case BFD_RELOC_RVA: case BFD_RELOC_RVA:
case BFD_RELOC_32: case BFD_RELOC_32:
case BFD_RELOC_16:
/* If we are deleting this reloc entry, we must fill in the /* If we are deleting this reloc entry, we must fill in the
value now. This can happen if we have a .word which is not value now. This can happen if we have a .word which is not
resolved when it appears but is later defined. */ resolved when it appears but is later defined. */
if (fixP->fx_done) if (fixP->fx_done)
md_number_to_chars ((char *) buf, *valP, 4); md_number_to_chars ((char *) buf, *valP, fixP->fx_size);
break;
case BFD_RELOC_16:
/* If we are deleting this reloc entry, we must fill in the
value now. */
if (fixP->fx_done)
md_number_to_chars ((char *) buf, *valP, 2);
break; break;
case BFD_RELOC_LO16: case BFD_RELOC_LO16:
@ -11852,15 +11846,13 @@ md_apply_fix (fixS *fixP, valueT *valP, segT seg ATTRIBUTE_UNUSED)
as_bad_where (fixP->fx_file, fixP->fx_line, as_bad_where (fixP->fx_file, fixP->fx_line,
_("Branch to misaligned address (%lx)"), (long) *valP); _("Branch to misaligned address (%lx)"), (long) *valP);
/* /* We need to save the bits in the instruction since fixup_segment()
* We need to save the bits in the instruction since fixup_segment() might be deleting the relocation entry (i.e., a branch within
* might be deleting the relocation entry (i.e., a branch within the current segment). */
* the current segment).
*/
if (! fixP->fx_done) if (! fixP->fx_done)
break; break;
/* update old instruction data */ /* Update old instruction data. */
if (target_big_endian) if (target_big_endian)
insn = (buf[0] << 24) | (buf[1] << 16) | (buf[2] << 8) | buf[3]; insn = (buf[0] << 24) | (buf[1] << 16) | (buf[2] << 8) | buf[3];
else else
@ -11964,17 +11956,13 @@ s_align (int x ATTRIBUTE_UNUSED)
long temp_fill; long temp_fill;
long max_alignment = 15; long max_alignment = 15;
/* /* o Note that the assembler pulls down any immediately preceding label
o Note that the assembler pulls down any immediately preceding label
to the aligned address. to the aligned address.
o It's not documented but auto alignment is reinstated by o It's not documented but auto alignment is reinstated by
a .align pseudo instruction. a .align pseudo instruction.
o Note also that after auto alignment is turned off the mips assembler o Note also that after auto alignment is turned off the mips assembler
issues an error on attempt to assemble an improperly aligned data item. issues an error on attempt to assemble an improperly aligned data item.
We don't. We don't. */
*/
temp = get_absolute_expression (); temp = get_absolute_expression ();
if (temp > max_alignment) if (temp > max_alignment)
@ -11995,7 +11983,7 @@ s_align (int x ATTRIBUTE_UNUSED)
{ {
segment_info_type *si = seg_info (now_seg); segment_info_type *si = seg_info (now_seg);
struct insn_label_list *l = si->label_list; struct insn_label_list *l = si->label_list;
/* Auto alignment should be switched on by next section change */ /* Auto alignment should be switched on by next section change. */
auto_align = 1; auto_align = 1;
mips_align (temp, (int) temp_fill, l != NULL ? l->label : NULL); mips_align (temp, (int) temp_fill, l != NULL ? l->label : NULL);
} }
@ -12635,8 +12623,7 @@ s_cpload (int ignore ATTRIBUTE_UNUSED)
The -mno-shared option replaces the last three instructions with The -mno-shared option replaces the last three instructions with
lui $gp,%hi(_gp) lui $gp,%hi(_gp)
addiu $gp,$gp,%lo(_gp) addiu $gp,$gp,%lo(_gp) */
*/
static void static void
s_cpsetup (int ignore ATTRIBUTE_UNUSED) s_cpsetup (int ignore ATTRIBUTE_UNUSED)
@ -12787,8 +12774,8 @@ s_cprestore (int ignore ATTRIBUTE_UNUSED)
ld $gp, offset($sp) ld $gp, offset($sp)
If a register $reg2 was given there, it results in: If a register $reg2 was given there, it results in:
daddu $gp, $reg2, $0 daddu $gp, $reg2, $0 */
*/
static void static void
s_cpreturn (int ignore ATTRIBUTE_UNUSED) s_cpreturn (int ignore ATTRIBUTE_UNUSED)
{ {
@ -12982,8 +12969,7 @@ s_mips_stab (int type)
s_stab (type); s_stab (type);
} }
/* Handle the .weakext pseudo-op as defined in Kane and Heinrich. /* Handle the .weakext pseudo-op as defined in Kane and Heinrich. */
*/
static void static void
s_mips_weakext (int ignore ATTRIBUTE_UNUSED) s_mips_weakext (int ignore ATTRIBUTE_UNUSED)
@ -13728,7 +13714,7 @@ md_convert_frag (bfd *abfd ATTRIBUTE_UNUSED, segT asec, fragS *fragp)
md_number_to_chars ((char *) buf, insn, 4); md_number_to_chars ((char *) buf, insn, 4);
buf += 4; buf += 4;
/* Nop */ /* nop */
md_number_to_chars ((char *) buf, 0, 4); md_number_to_chars ((char *) buf, 0, 4);
buf += 4; buf += 4;
@ -14188,7 +14174,7 @@ md_obj_begin (void)
static void static void
md_obj_end (void) md_obj_end (void)
{ {
/* check for premature end, nesting errors, etc */ /* Check for premature end, nesting errors, etc. */
if (cur_proc_ptr) if (cur_proc_ptr)
as_warn (_("missing .end at end of assembly")); as_warn (_("missing .end at end of assembly"));
} }