espressif/binutils-gdb
1
0
Fork 0
mirror of https://github.com/espressif/binutils-gdb.git synced 2025-06-25 04:49:54 +08:00
Code Issues Packages Projects Releases Wiki Activity

PR gas/5155

Browse Source 
* config/tc-msp430.c: Fix spelling typos.
This commit is contained in:
Nick Clifton
2007-10-11 13:48:39 +00:00
parent 5671778d0a
commit 79cf59509d
2 changed files with 22 additions and 20 deletions
Download Patch File Download Diff File Expand all files Collapse all files

5
gas/ChangeLog
View File

@ -1,3 +1,8 @@
2007-10-11 Nick Clifton <nickc@redhat.com>
PR gas/5155
* config/tc-msp430.c: Fix spelling typos.
2007-10-11 Nick Clifton <nickc@redhat.com> 2007-10-11 Nick Clifton <nickc@redhat.com>
* doc/as.texinfo (Type): Fix typo: STT_FUNC not STT_FUNCTION. * doc/as.texinfo (Type): Fix typo: STT_FUNC not STT_FUNCTION.

37
gas/config/tc-msp430.c
View File

@ -30,8 +30,7 @@
#include "safe-ctype.h" #include "safe-ctype.h"
#include "dwarf2dbg.h" #include "dwarf2dbg.h"
/* /* We will disable polymorphs by default because it is dangerous.
We will disable polymorphs by default because it is dangerous.
The potential problem here is the following: assume we got the The potential problem here is the following: assume we got the
following code: following code:
@ -50,8 +49,8 @@
8: nop 8: nop
10: ret 10: ret
If the 'subroutine' wiys thin +-1024 bytes range then linker If the 'subroutine' is within +-1024 bytes range then linker
will produce will produce:
0: jmp .text +0x08 0: jmp .text +0x08
2: nop 2: nop
4: jmp subroutine 4: jmp subroutine
@ -59,16 +58,14 @@
6: nop 6: nop
8: ret ; 'jmp .text +0x08' will land here. WRONG!!! 8: ret ; 'jmp .text +0x08' will land here. WRONG!!!
The workaround is the following: The workaround is the following:
1. Declare global var enable_polymorphs which set to 1 via option -mP. 1. Declare global var enable_polymorphs which set to 1 via option -mp.
2. Declare global var enable_relax which set to 1 via option -mQ. 2. Declare global var enable_relax which set to 1 via option -mQ.
If polymorphs are enabled, and relax isn't, treat all jumps as long jumps, If polymorphs are enabled, and relax isn't, treat all jumps as long jumps,
do not delete any relocs and leave them for linker. do not delete any relocs and leave them for linker.
If relax is enabled, relax at assembly time and kill relocs as necessary. If relax is enabled, relax at assembly time and kill relocs as necessary. */
*/
int msp430_enable_relax; int msp430_enable_relax;
int msp430_enable_polys; int msp430_enable_polys;
@ -394,7 +391,7 @@ static struct mcu_type_s * msp430_mcu = & default_mcu;
push r8 push r8
.profiler "cdp",fxx,0, .LFrameOffset_fxx ; check stack value at this point .profiler "cdp",fxx,0, .LFrameOffset_fxx ; check stack value at this point
; (this is a prologue end) ; (this is a prologue end)
; note, that spare var filled with the farme size ; note, that spare var filled with the frame size
mov r15,r8 mov r15,r8
.... ....
.profiler cdE,fxx ; check stack .profiler cdE,fxx ; check stack
@ -622,17 +619,17 @@ msp430_profiler (int dummy ATTRIBUTE_UNUSED)
|| ! pow2value (p_flags & ( MSP430_PROFILER_FLAG_INITSECT || ! pow2value (p_flags & ( MSP430_PROFILER_FLAG_INITSECT
| MSP430_PROFILER_FLAG_FINISECT)))) | MSP430_PROFILER_FLAG_FINISECT))))
{ {
as_bad (_("ambigious flags combination - '.profiler' directive ignored.")); as_bad (_("ambiguous flags combination - '.profiler' directive ignored."));
input_line_pointer = end; input_line_pointer = end;
return; return;
} }
/* Generate temp symbol which denotes current location. */ /* Generate temp symbol which denotes current location. */
if (now_seg == absolute_section) /* Paranoja ? */ if (now_seg == absolute_section) /* Paranoia ? */
{ {
exp1.X_op = O_constant; exp1.X_op = O_constant;
exp1.X_add_number = abs_section_offset; exp1.X_add_number = abs_section_offset;
as_warn (_("profiling in absolute section? Hm...")); as_warn (_("profiling in absolute section?"));
} }
else else
{ {
@ -1142,7 +1139,7 @@ msp430_srcoperand (struct msp430_operand_s * op,
op->mode = OP_REG; op->mode = OP_REG;
} }
} }
/* Redudant (yet) check. */ /* Redundant (yet) check. */
else if (op->exp.X_op == O_register) else if (op->exp.X_op == O_register)
as_bad as_bad
(_("Registers cannot be used within immediate expression [%s]"), l); (_("Registers cannot be used within immediate expression [%s]"), l);
@ -1177,7 +1174,7 @@ msp430_srcoperand (struct msp430_operand_s * op,
; ;
else else
{ {
/* Redudant (yet) check. */ /* Redundant (yet) check. */
if (op->exp.X_op == O_register) if (op->exp.X_op == O_register)
as_bad as_bad
(_("Registers cannot be used within absolute expression [%s]"), l); (_("Registers cannot be used within absolute expression [%s]"), l);
@ -1265,7 +1262,7 @@ msp430_srcoperand (struct msp430_operand_s * op,
op->reg = *t - '0'; op->reg = *t - '0';
if (op->reg > 9 || op->reg < 0) if (op->reg > 9 || op->reg < 0)
{ {
as_bad (_("unknown operator (r%s substituded as a register name"), as_bad (_("unknown operator (r%s substituted as a register name"),
t); t);
return 1; return 1;
} }
@ -1320,7 +1317,7 @@ msp430_srcoperand (struct msp430_operand_s * op,
; ;
else else
{ {
/* Redudant (yet) check. */ /* Redundant (yet) check. */
if (op->exp.X_op == O_register) if (op->exp.X_op == O_register)
as_bad as_bad
(_("Registers cannot be used as a prefix of indexed expression [%s]"), l); (_("Registers cannot be used as a prefix of indexed expression [%s]"), l);
@ -1831,7 +1828,7 @@ msp430_operands (struct msp430_opcode_s * opcode, char * line)
break; break;
default: default:
as_bad (_("Ilegal instruction or not implmented opcode.")); as_bad (_("Illegal instruction or not implemented opcode."));
} }
input_line_pointer = line; input_line_pointer = line;
@ -1987,7 +1984,7 @@ md_apply_fix (fixS * fixp, valueT * valuep, segT seg)
{ {
if (!fixp->fx_addsy || (fixp->fx_addsy if (!fixp->fx_addsy || (fixp->fx_addsy
&& S_GET_SEGMENT (fixp->fx_addsy) == absolute_section)) && S_GET_SEGMENT (fixp->fx_addsy) == absolute_section))
fixp->fx_done = 1; /* it is ok to kill 'abs' reloc */ fixp->fx_done = 1; /* It is ok to kill 'abs' reloc. */
else else
fixp->fx_done = 0; fixp->fx_done = 0;
} }
@ -2120,7 +2117,7 @@ md_estimate_size_before_relax (fragS * fragP ATTRIBUTE_UNUSED,
else if (fragP->fr_symbol) else if (fragP->fr_symbol)
{ {
/* Its got a segment, but its not ours. Even if fr_symbol is in /* Its got a segment, but its not ours. Even if fr_symbol is in
an absolute segment, we dont know a displacement until we link an absolute segment, we don't know a displacement until we link
object files. So it will always be long. This also applies to object files. So it will always be long. This also applies to
labels in a subsegment of current. Liker may relax it to short labels in a subsegment of current. Liker may relax it to short
jump later. Return value == 8. */ jump later. Return value == 8. */
@ -2130,7 +2127,7 @@ md_estimate_size_before_relax (fragS * fragP ATTRIBUTE_UNUSED,
else else
{ {
/* We know the abs value. may be it is a jump to fixed address. /* We know the abs value. may be it is a jump to fixed address.
Impossible in our case, cause all constants already handeled. */ Impossible in our case, cause all constants already handled. */
fragP->fr_subtype = fragP->fr_subtype =
ENCODE_RELAX (RELAX_LEN (fragP->fr_subtype), STATE_UNDEF); ENCODE_RELAX (RELAX_LEN (fragP->fr_subtype), STATE_UNDEF);
} }