OpenE2K
/
binutils-gdb
12
0
Fork 0
Code Issues Pull Requests Releases Wiki Activity

Fix compile time warning messages about signed/unsigned conflicts reported 

by GCC 4.0
This commit is contained in:
Nick Clifton 2005-07-04 14:27:07 +00:00
parent 80447b21a9
commit ca3f61f77c
2 changed files with 30 additions and 17 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

13
gas/ChangeLog
View File

@ -1,3 +1,16 @@
2005-07-04 Khem Raj <kraj@mvista.com>
Nick Clifton <nickc@redhat.com>
* tc-arm.c (struct arm_it): Make operands.imm signed to match its
use an immediate value.
(parse_vfp_reg_list): Make the 2nd parameter an unsigned pointer
since the register field of the operands structure is unsigned.
(s_arm_unwind_save_vfp): Make "reg" unsigned.
(parse_operands): Make the 2ns parameter an unsigned pointer to
match its use.
(do_ldrd): When using the imm field of the operands structure as a
second register field, treat it as unsigned.
2005-07-04 Alan Modra <amodra@bigpond.net.au> 2005-07-04 Alan Modra <amodra@bigpond.net.au>
PR 1004 PR 1004

34
gas/config/tc-arm.c
View File

@ -226,17 +226,17 @@ struct arm_it
struct struct
{ {
unsigned reg; unsigned reg;
unsigned imm; signed int imm;
unsigned present : 1; /* operand present */ unsigned present : 1; /* Operand present. */
unsigned isreg : 1; /* operand was a register */ unsigned isreg : 1; /* Operand was a register. */
unsigned immisreg : 1; /* .imm field is a second register */ unsigned immisreg : 1; /* .imm field is a second register. */
unsigned hasreloc : 1; /* operand has relocation suffix */ unsigned hasreloc : 1; /* Operand has relocation suffix. */
unsigned writeback : 1; /* operand has trailing ! */ unsigned writeback : 1; /* Operand has trailing ! */
unsigned preind : 1; /* preindexed address */ unsigned preind : 1; /* Preindexed address. */
unsigned postind : 1; /* postindexed address */ unsigned postind : 1; /* Postindexed address. */
unsigned negative : 1; /* index register was negated */ unsigned negative : 1; /* Index register was negated. */
unsigned shifted : 1; /* shift applied to operation */ unsigned shifted : 1; /* Shift applied to operation. */
unsigned shift_kind : 3; /* shift operation (enum shift_kind) */ unsigned shift_kind : 3; /* Shift operation (enum shift_kind). */
} operands[6]; } operands[6];
}; };
@ -1035,7 +1035,7 @@ parse_reg_list (char ** strp)
register. Double precision registers are matched if DP is nonzero. */ register. Double precision registers are matched if DP is nonzero. */
static int static int
parse_vfp_reg_list (char **str, int *pbase, int dp) parse_vfp_reg_list (char **str, unsigned int *pbase, int dp)
{ {
int base_reg; int base_reg;
int new_base; int new_base;
@ -2335,7 +2335,7 @@ static void
s_arm_unwind_save_vfp (void) s_arm_unwind_save_vfp (void)
{ {
int count; int count;
int reg; unsigned int reg;
valueT op; valueT op;
count = parse_vfp_reg_list (&input_line_pointer, &reg, 1); count = parse_vfp_reg_list (&input_line_pointer, &reg, 1);
@ -3246,7 +3246,7 @@ parse_address (char **str, int i)
{ {
/* [Rn], {expr} - unindexed, with option */ /* [Rn], {expr} - unindexed, with option */
if (parse_immediate (&p, &inst.operands[i].imm, if (parse_immediate (&p, &inst.operands[i].imm,
0, 255, TRUE) == FAIL) 0, 255, TRUE) == FAIL)
return FAIL; return FAIL;
if (skip_past_char (&p, '}') == FAIL) if (skip_past_char (&p, '}') == FAIL)
@ -3574,7 +3574,7 @@ enum operand_parse_code
structure. Returns SUCCESS or FAIL depending on whether the structure. Returns SUCCESS or FAIL depending on whether the
specified grammar matched. */ specified grammar matched. */
static int static int
parse_operands (char *str, const char *pattern) parse_operands (char *str, const unsigned char *pattern)
{ {
unsigned const char *upat = pattern; unsigned const char *upat = pattern;
char *backtrack_pos = 0; char *backtrack_pos = 0;
@ -4645,8 +4645,8 @@ do_ldrd (void)
/* For an index-register load, the index register must not overlap the /* For an index-register load, the index register must not overlap the
destination (even if not write-back). */ destination (even if not write-back). */
else if (inst.operands[2].immisreg else if (inst.operands[2].immisreg
&& (inst.operands[2].imm == inst.operands[0].reg && ((unsigned) inst.operands[2].imm == inst.operands[0].reg
|| inst.operands[2].imm == inst.operands[1].reg)) || (unsigned) inst.operands[2].imm == inst.operands[1].reg))
as_warn (_("index register overlaps destination register")); as_warn (_("index register overlaps destination register"));
} }