From 79cf59509dc438a60749ff82ba522376197cd56d Mon Sep 17 00:00:00 2001 From: Nick Clifton Date: Thu, 11 Oct 2007 13:48:39 +0000 Subject: [PATCH] PR gas/5155 * config/tc-msp430.c: Fix spelling typos. --- gas/ChangeLog | 5 +++++ gas/config/tc-msp430.c | 37 +++++++++++++++++-------------------- 2 files changed, 22 insertions(+), 20 deletions(-) diff --git a/gas/ChangeLog b/gas/ChangeLog index 500beb23126..fb69c95a692 100644 --- a/gas/ChangeLog +++ b/gas/ChangeLog @@ -1,3 +1,8 @@ +2007-10-11 Nick Clifton + + PR gas/5155 + * config/tc-msp430.c: Fix spelling typos. + 2007-10-11 Nick Clifton * doc/as.texinfo (Type): Fix typo: STT_FUNC not STT_FUNCTION. diff --git a/gas/config/tc-msp430.c b/gas/config/tc-msp430.c index 7b248e614bf..27cbe38ecbc 100644 --- a/gas/config/tc-msp430.c +++ b/gas/config/tc-msp430.c @@ -30,8 +30,7 @@ #include "safe-ctype.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 following code: @@ -50,8 +49,8 @@ 8: nop 10: ret - If the 'subroutine' wiys thin +-1024 bytes range then linker - will produce + If the 'subroutine' is within +-1024 bytes range then linker + will produce: 0: jmp .text +0x08 2: nop 4: jmp subroutine @@ -59,16 +58,14 @@ 6: nop 8: ret ; 'jmp .text +0x08' will land here. WRONG!!! - 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. If polymorphs are enabled, and relax isn't, treat all jumps as long jumps, 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_polys; @@ -394,7 +391,7 @@ static struct mcu_type_s * msp430_mcu = & default_mcu; push r8 .profiler "cdp",fxx,0, .LFrameOffset_fxx ; check stack value at this point ; (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 .... .profiler cdE,fxx ; check stack @@ -622,17 +619,17 @@ msp430_profiler (int dummy ATTRIBUTE_UNUSED) || ! pow2value (p_flags & ( MSP430_PROFILER_FLAG_INITSECT | MSP430_PROFILER_FLAG_FINISECT)))) { - as_bad (_("ambigious flags combination - '.profiler' directive ignored.")); + as_bad (_("ambiguous flags combination - '.profiler' directive ignored.")); input_line_pointer = end; return; } /* 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_add_number = abs_section_offset; - as_warn (_("profiling in absolute section? Hm...")); + as_warn (_("profiling in absolute section?")); } else { @@ -1142,7 +1139,7 @@ msp430_srcoperand (struct msp430_operand_s * op, op->mode = OP_REG; } } - /* Redudant (yet) check. */ + /* Redundant (yet) check. */ else if (op->exp.X_op == O_register) as_bad (_("Registers cannot be used within immediate expression [%s]"), l); @@ -1177,7 +1174,7 @@ msp430_srcoperand (struct msp430_operand_s * op, ; else { - /* Redudant (yet) check. */ + /* Redundant (yet) check. */ if (op->exp.X_op == O_register) as_bad (_("Registers cannot be used within absolute expression [%s]"), l); @@ -1265,7 +1262,7 @@ msp430_srcoperand (struct msp430_operand_s * op, op->reg = *t - '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); return 1; } @@ -1320,7 +1317,7 @@ msp430_srcoperand (struct msp430_operand_s * op, ; else { - /* Redudant (yet) check. */ + /* Redundant (yet) check. */ if (op->exp.X_op == O_register) as_bad (_("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; default: - as_bad (_("Ilegal instruction or not implmented opcode.")); + as_bad (_("Illegal instruction or not implemented opcode.")); } input_line_pointer = line; @@ -1987,7 +1984,7 @@ md_apply_fix (fixS * fixp, valueT * valuep, segT seg) { if (!fixp->fx_addsy || (fixp->fx_addsy && 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 fixp->fx_done = 0; } @@ -2120,7 +2117,7 @@ md_estimate_size_before_relax (fragS * fragP ATTRIBUTE_UNUSED, else if (fragP->fr_symbol) { /* 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 labels in a subsegment of current. Liker may relax it to short jump later. Return value == 8. */ @@ -2130,7 +2127,7 @@ md_estimate_size_before_relax (fragS * fragP ATTRIBUTE_UNUSED, else { /* 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 = ENCODE_RELAX (RELAX_LEN (fragP->fr_subtype), STATE_UNDEF); } -- 2.30.2