diff options
| author | Nick Clifton <nickc@redhat.com> | 2009-12-11 13:41:59 +0000 |
|---|---|---|
| committer | Nick Clifton <nickc@redhat.com> | 2009-12-11 13:41:59 +0000 |
| commit | 5413b5486ba77949122a8bfd6067d31626e330f3 (patch) | |
| tree | 6956d71f05d79d40e52fed28668bf9db104a4a04 /bfd/elf32-xtensa.c | |
| parent | f28844f209b0de6783df75e402dd1c26f3edb0c7 (diff) | |
| download | binutils-redhat-5413b5486ba77949122a8bfd6067d31626e330f3.tar.gz | |
Add -Wshadow to the gcc command line options used when compiling the binutils.
Fix up all warnings generated by the addition of this switch.
Diffstat (limited to 'bfd/elf32-xtensa.c')
| -rw-r--r-- | bfd/elf32-xtensa.c | 19 |
1 files changed, 9 insertions, 10 deletions
diff --git a/bfd/elf32-xtensa.c b/bfd/elf32-xtensa.c index 9f961dcde2..1740c355a6 100644 --- a/bfd/elf32-xtensa.c +++ b/bfd/elf32-xtensa.c @@ -1943,9 +1943,8 @@ elf_xtensa_do_reloc (reloc_howto_type *howto, if (!is_weak_undef) { /* Check for windowed CALL across a 1GB boundary. */ - xtensa_opcode opcode = - get_expanded_call_opcode (contents + address, - input_size - address, 0); + opcode = get_expanded_call_opcode (contents + address, + input_size - address, 0); if (is_windowed_call_opcode (opcode)) { if ((self_address >> CALL_SEGMENT_BITS) @@ -2665,7 +2664,7 @@ elf_xtensa_relocate_section (bfd *output_bfd, if (r_type == R_XTENSA_ASM_SIMPLIFY) { - char *error_message = NULL; + error_message = NULL; /* Convert ASM_SIMPLIFY into the simpler relocation so that they never escape a relaxing link. */ r = contract_asm_expansion (contents, input_size, rel, @@ -3166,18 +3165,18 @@ elf_xtensa_combine_prop_entries (bfd *output_bfd, for (n = 0; n < num; n++) { - bfd_boolean remove = FALSE; + bfd_boolean remove_entry = FALSE; if (table[n].size == 0) - remove = TRUE; - else if (n > 0 && - (table[n-1].address + table[n-1].size == table[n].address)) + remove_entry = TRUE; + else if (n > 0 + && (table[n-1].address + table[n-1].size == table[n].address)) { table[n-1].size += table[n].size; - remove = TRUE; + remove_entry = TRUE; } - if (remove) + if (remove_entry) { for (m = n; m < num - 1; m++) { |