From 91d6fa6a035cc7d0b7be5c99c194a64cb80924b0 Mon Sep 17 00:00:00 2001 From: Nick Clifton Date: Fri, 11 Dec 2009 13:42:17 +0000 Subject: Add -Wshadow to the gcc command line options used when compiling the binutils. Fix up all warnings generated by the addition of this switch. --- gold/object.cc | 316 ++++++++++++++++++++++++++++----------------------------- 1 file changed, 158 insertions(+), 158 deletions(-) (limited to 'gold/object.cc') diff --git a/gold/object.cc b/gold/object.cc index 798e42d..465e99c 100644 --- a/gold/object.cc +++ b/gold/object.cc @@ -51,12 +51,12 @@ namespace gold template void -Xindex::initialize_symtab_xindex(Object* object, unsigned int symtab_shndx) +Xindex::initialize_symtab_xindex(Object* object, unsigned int sym_tab_shndx) { if (!this->symtab_xindex_.empty()) return; - gold_assert(symtab_shndx != 0); + gold_assert(sym_tab_shndx != 0); // Look through the sections in reverse order, on the theory that it // is more likely to be near the end than the beginning. @@ -65,7 +65,7 @@ Xindex::initialize_symtab_xindex(Object* object, unsigned int symtab_shndx) { --i; if (object->section_type(i) == elfcpp::SHT_SYMTAB_SHNDX - && this->adjust_shndx(object->section_link(i)) == symtab_shndx) + && this->adjust_shndx(object->section_link(i)) == sym_tab_shndx) { this->read_symtab_xindex(object, i, NULL); return; @@ -177,8 +177,8 @@ Object::read_section_data(elfcpp::Elf_file* elf_file, // Read the section headers. const off_t shoff = elf_file->shoff(); - const unsigned int shnum = this->shnum(); - sd->section_headers = this->get_lasting_view(shoff, shnum * shdr_size, + const unsigned int sec_shnum = this->shnum(); + sd->section_headers = this->get_lasting_view(shoff, sec_shnum * shdr_size, true, true); // Read the section names. @@ -197,17 +197,17 @@ Object::read_section_data(elfcpp::Elf_file* elf_file, false); } -// If NAME is the name of a special .gnu.warning section, arrange for +// If SNAME is the name of a special .gnu.warning section, arrange for // the warning to be issued. SHNDX is the section index. Return // whether it is a warning section. bool -Object::handle_gnu_warning_section(const char* name, unsigned int shndx, +Object::handle_gnu_warning_section(const char* sname, unsigned int shndx, Symbol_table* symtab) { const char warn_prefix[] = ".gnu.warning."; const int warn_prefix_len = sizeof warn_prefix - 1; - if (strncmp(name, warn_prefix, warn_prefix_len) == 0) + if (strncmp(sname, warn_prefix, warn_prefix_len) == 0) { // Read the section contents to get the warning text. It would // be nicer if we only did this if we have to actually issue a @@ -220,29 +220,29 @@ Object::handle_gnu_warning_section(const char* name, unsigned int shndx, false); if (len == 0) { - const char* warning = name + warn_prefix_len; + const char* warning = sname + warn_prefix_len; contents = reinterpret_cast(warning); len = strlen(warning); } std::string warning(reinterpret_cast(contents), len); - symtab->add_warning(name + warn_prefix_len, this, warning); + symtab->add_warning(sname + warn_prefix_len, this, warning); return true; } return false; } -// If NAME is the name of the special section which indicates that +// If SNAME is the name of the special section which indicates that // this object was compiled with -fstack-split, mark it accordingly. bool -Object::handle_split_stack_section(const char* name) +Object::handle_split_stack_section(const char* sname) { - if (strcmp(name, ".note.GNU-split-stack") == 0) + if (strcmp(sname, ".note.GNU-split-stack") == 0) { this->uses_split_stack_ = true; return true; } - if (strcmp(name, ".note.GNU-no-split-stack") == 0) + if (strcmp(sname, ".note.GNU-no-split-stack") == 0) { this->has_no_split_stack_ = true; return true; @@ -301,22 +301,22 @@ Relobj::copy_symbols_data(Symbols_data* gc_sd, Read_symbols_data* sd, // roots of the worklist. bool -Relobj::is_section_name_included(const char* name) +Relobj::is_section_name_included(const char* sname) { - if (is_prefix_of(".ctors", name) - || is_prefix_of(".dtors", name) - || is_prefix_of(".note", name) - || is_prefix_of(".init", name) - || is_prefix_of(".fini", name) - || is_prefix_of(".gcc_except_table", name) - || is_prefix_of(".jcr", name) - || is_prefix_of(".preinit_array", name) - || (is_prefix_of(".text", name) - && strstr(name, "personality")) - || (is_prefix_of(".data", name) - && strstr(name, "personality")) - || (is_prefix_of(".gnu.linkonce.d", name) && - strstr(name, "personality"))) + if (is_prefix_of(".ctors", sname) + || is_prefix_of(".dtors", sname) + || is_prefix_of(".note", sname) + || is_prefix_of(".init", sname) + || is_prefix_of(".fini", sname) + || is_prefix_of(".gcc_except_table", sname) + || is_prefix_of(".jcr", sname) + || is_prefix_of(".preinit_array", sname) + || (is_prefix_of(".text", sname) + && strstr(sname, "personality")) + || (is_prefix_of(".data", sname) + && strstr(sname, "personality")) + || (is_prefix_of(".gnu.linkonce.d", sname) + && strstr(sname, "personality"))) { return true; } @@ -327,11 +327,11 @@ Relobj::is_section_name_included(const char* name) template Sized_relobj::Sized_relobj( - const std::string& name, - Input_file* input_file, - off_t offset, + const std::string& aname, + Input_file* ainput_file, + off_t aoffset, const elfcpp::Ehdr& ehdr) - : Relobj(name, input_file, offset), + : Relobj(aname, ainput_file, aoffset), elf_file_(this, ehdr), symtab_shndx_(-1U), local_symbol_count_(0), @@ -361,8 +361,8 @@ template void Sized_relobj::do_setup() { - const unsigned int shnum = this->elf_file_.shnum(); - this->set_shnum(shnum); + const unsigned int sec_shnum = this->elf_file_.shnum(); + this->set_shnum(sec_shnum); } // Find the SHT_SYMTAB section, given the section headers. The ELF @@ -374,14 +374,14 @@ template void Sized_relobj::find_symtab(const unsigned char* pshdrs) { - const unsigned int shnum = this->shnum(); + const unsigned int sec_shnum = this->shnum(); this->symtab_shndx_ = 0; - if (shnum > 0) + if (sec_shnum > 0) { // Look through the sections in reverse order, since gas tends // to put the symbol table at the end. - const unsigned char* p = pshdrs + shnum * This::shdr_size; - unsigned int i = shnum; + const unsigned char* p = pshdrs + sec_shnum * This::shdr_size; + unsigned int i = sec_shnum; unsigned int xindex_shndx = 0; unsigned int xindex_link = 0; while (i > 0) @@ -451,9 +451,9 @@ Sized_relobj::find_eh_frame( const char* names, section_size_type names_size) const { - const unsigned int shnum = this->shnum(); + const unsigned int sec_shnum = this->shnum(); const unsigned char* p = pshdrs + This::shdr_size; - for (unsigned int i = 1; i < shnum; ++i, p += This::shdr_size) + for (unsigned int i = 1; i < sec_shnum; ++i, p += This::shdr_size) { typename This::Shdr shdr(p); if (this->check_eh_frame_flags(&shdr)) @@ -465,8 +465,8 @@ Sized_relobj::find_eh_frame( continue; } - const char* name = names + shdr.get_sh_name(); - if (strcmp(name, ".eh_frame") == 0) + const char* aname = names + shdr.get_sh_name(); + if (strcmp(aname, ".eh_frame") == 0) return true; } } @@ -517,11 +517,11 @@ Sized_relobj::do_read_symbols(Read_symbols_data* sd) // object file format is over 5 pages that we don't need to read // now. - const int sym_size = This::sym_size; + const int symsize = This::sym_size; const unsigned int loccount = symtabshdr.get_sh_info(); this->local_symbol_count_ = loccount; this->local_values_.resize(loccount); - section_offset_type locsize = loccount * sym_size; + section_offset_type locsize = loccount * symsize; off_t dataoff = symtabshdr.get_sh_offset(); section_size_type datasize = convert_to_section_size_type(symtabshdr.get_sh_size()); @@ -604,9 +604,9 @@ template bool Sized_relobj::include_section_group( Symbol_table* symtab, - Layout* layout, + Layout* alayout, unsigned int index, - const char* name, + const char* aname, const unsigned char* shdrs, const char* section_names, section_size_type section_names_size, @@ -697,9 +697,9 @@ Sized_relobj::include_section_group( } else { - include_group = layout->find_or_add_kept_section(signature, - this, index, true, - true, &kept_section); + include_group = alayout->find_or_add_kept_section(signature, + this, index, true, + true, &kept_section); is_comdat = true; } @@ -792,8 +792,8 @@ Sized_relobj::include_section_group( } if (relocate_group) - layout->layout_group(symtab, this, index, name, signature.c_str(), - shdr, flags, &shndxes); + alayout->layout_group(symtab, this, index, aname, signature.c_str(), + shdr, flags, &shndxes); return include_group; } @@ -816,9 +816,9 @@ Sized_relobj::include_section_group( template bool Sized_relobj::include_linkonce_section( - Layout* layout, + Layout* alayout, unsigned int index, - const char* name, + const char* aname, const elfcpp::Shdr& shdr) { typename elfcpp::Elf_types::Elf_WXword sh_size = shdr.get_sh_size(); @@ -832,18 +832,18 @@ Sized_relobj::include_linkonce_section( // ".gnu.linkonce.d.rel.ro.local". const char* const linkonce_t = ".gnu.linkonce.t."; const char* symname; - if (strncmp(name, linkonce_t, strlen(linkonce_t)) == 0) - symname = name + strlen(linkonce_t); + if (strncmp(aname, linkonce_t, strlen(linkonce_t)) == 0) + symname = aname + strlen(linkonce_t); else - symname = strrchr(name, '.') + 1; + symname = strrchr(aname, '.') + 1; std::string sig1(symname); - std::string sig2(name); + std::string sig2(aname); Kept_section* kept1; Kept_section* kept2; - bool include1 = layout->find_or_add_kept_section(sig1, this, index, false, - false, &kept1); - bool include2 = layout->find_or_add_kept_section(sig2, this, index, false, - true, &kept2); + bool include1 = alayout->find_or_add_kept_section(sig1, this, index, false, + false, &kept1); + bool include2 = alayout->find_or_add_kept_section(sig2, this, index, false, + true, &kept2); if (!include2) { @@ -887,27 +887,27 @@ Sized_relobj::include_linkonce_section( template inline void -Sized_relobj::layout_section(Layout* layout, +Sized_relobj::layout_section(Layout* alayout, unsigned int shndx, - const char* name, + const char* aname, typename This::Shdr& shdr, unsigned int reloc_shndx, unsigned int reloc_type) { - off_t offset; - Output_section* os = layout->layout(this, shndx, name, shdr, - reloc_shndx, reloc_type, &offset); + off_t aoffset; + Output_section* os = alayout->layout(this, shndx, aname, shdr, + reloc_shndx, reloc_type, &aoffset); this->output_sections()[shndx] = os; - if (offset == -1) + if (aoffset == -1) this->section_offsets_[shndx] = invalid_address; else - this->section_offsets_[shndx] = convert_types(offset); + this->section_offsets_[shndx] = convert_types(aoffset); // If this section requires special handling, and if there are // relocs that apply to it, then we must do the special handling // before we apply the relocs. - if (offset == -1 && reloc_shndx != 0) + if (aoffset == -1 && reloc_shndx != 0) this->set_relocs_must_follow_section_writes(); } @@ -928,10 +928,10 @@ Sized_relobj::layout_section(Layout* layout, template void Sized_relobj::do_layout(Symbol_table* symtab, - Layout* layout, + Layout* alayout, Read_symbols_data* sd) { - const unsigned int shnum = this->shnum(); + const unsigned int sec_shnum = this->shnum(); bool is_gc_pass_one = ((parameters->options().gc_sections() && !symtab->gc()->is_worklist_ready()) || (parameters->options().icf_enabled() @@ -948,7 +948,7 @@ Sized_relobj::do_layout(Symbol_table* symtab, // Both is_gc_pass_one and is_gc_pass_two should not be true. gold_assert(!(is_gc_pass_one && is_gc_pass_two)); - if (shnum == 0) + if (sec_shnum == 0) return; Symbols_data* gc_sd = NULL; if (is_gc_pass_one) @@ -956,7 +956,7 @@ Sized_relobj::do_layout(Symbol_table* symtab, // During garbage collection save the symbols data to use it when // re-entering this function. gc_sd = new Symbols_data; - this->copy_symbols_data(gc_sd, sd, This::shdr_size * shnum); + this->copy_symbols_data(gc_sd, sd, This::shdr_size * sec_shnum); this->set_symbols_data(gc_sd); } else if (is_gc_pass_two) @@ -1016,11 +1016,11 @@ Sized_relobj::do_layout(Symbol_table* symtab, // For each section, record the index of the reloc section if any. // Use 0 to mean that there is no reloc section, -1U to mean that // there is more than one. - std::vector reloc_shndx(shnum, 0); - std::vector reloc_type(shnum, elfcpp::SHT_NULL); + std::vector reloc_shndx(sec_shnum, 0); + std::vector reloc_type(sec_shnum, elfcpp::SHT_NULL); // Skip the first, dummy, section. pshdrs = shdrs + This::shdr_size; - for (unsigned int i = 1; i < shnum; ++i, pshdrs += This::shdr_size) + for (unsigned int i = 1; i < sec_shnum; ++i, pshdrs += This::shdr_size) { typename This::Shdr shdr(pshdrs); @@ -1032,7 +1032,7 @@ Sized_relobj::do_layout(Symbol_table* symtab, if (sh_type == elfcpp::SHT_REL || sh_type == elfcpp::SHT_RELA) { unsigned int target_shndx = this->adjust_shndx(shdr.get_sh_info()); - if (target_shndx == 0 || target_shndx >= shnum) + if (target_shndx == 0 || target_shndx >= sec_shnum) { this->error(_("relocation section %u has bad info %u"), i, target_shndx); @@ -1054,8 +1054,8 @@ Sized_relobj::do_layout(Symbol_table* symtab, if (!is_gc_pass_two) { - out_sections.resize(shnum); - out_section_offsets.resize(shnum); + out_sections.resize(sec_shnum); + out_section_offsets.resize(sec_shnum); } // If we are only linking for symbols, then there is nothing else to @@ -1084,12 +1084,12 @@ Sized_relobj::do_layout(Symbol_table* symtab, uint64_t gnu_stack_flags = 0; // Keep track of which sections to omit. - std::vector omit(shnum, false); + std::vector omit(sec_shnum, false); // Keep track of reloc sections when emitting relocations. const bool relocatable = parameters->options().relocatable(); - const bool emit_relocs = (relocatable - || parameters->options().emit_relocs()); + const bool emit_rels = (relocatable + || parameters->options().emit_relocs()); std::vector reloc_sections; // Keep track of .eh_frame sections. @@ -1097,7 +1097,7 @@ Sized_relobj::do_layout(Symbol_table* symtab, // Skip the first, dummy, section. pshdrs = shdrs + This::shdr_size; - for (unsigned int i = 1; i < shnum; ++i, pshdrs += This::shdr_size) + for (unsigned int i = 1; i < sec_shnum; ++i, pshdrs += This::shdr_size) { typename This::Shdr shdr(pshdrs); @@ -1108,11 +1108,11 @@ Sized_relobj::do_layout(Symbol_table* symtab, return; } - const char* name = pnames + shdr.get_sh_name(); + const char* sname = pnames + shdr.get_sh_name(); if (!is_gc_pass_two) { - if (this->handle_gnu_warning_section(name, i, symtab)) + if (this->handle_gnu_warning_section(sname, i, symtab)) { if (!relocatable) omit[i] = true; @@ -1121,7 +1121,7 @@ Sized_relobj::do_layout(Symbol_table* symtab, // The .note.GNU-stack section is special. It gives the // protection flags that this object file requires for the stack // in memory. - if (strcmp(name, ".note.GNU-stack") == 0) + if (strcmp(sname, ".note.GNU-stack") == 0) { seen_gnu_stack = true; gnu_stack_flags |= shdr.get_sh_flags(); @@ -1131,7 +1131,7 @@ Sized_relobj::do_layout(Symbol_table* symtab, // The .note.GNU-split-stack section is also special. It // indicates that the object was compiled with // -fsplit-stack. - if (this->handle_split_stack_section(name)) + if (this->handle_split_stack_section(sname)) { if (!parameters->options().relocatable() && !parameters->options().shared()) @@ -1139,7 +1139,7 @@ Sized_relobj::do_layout(Symbol_table* symtab, } // Skip attributes section. - if (parameters->target().is_attributes_section(name)) + if (parameters->target().is_attributes_section(sname)) { omit[i] = true; } @@ -1149,16 +1149,16 @@ Sized_relobj::do_layout(Symbol_table* symtab, { if (shdr.get_sh_type() == elfcpp::SHT_GROUP) { - if (!this->include_section_group(symtab, layout, i, name, + if (!this->include_section_group(symtab, alayout, i, sname, shdrs, pnames, section_names_size, &omit)) discard = true; } else if ((shdr.get_sh_flags() & elfcpp::SHF_GROUP) == 0 - && Layout::is_linkonce(name)) + && Layout::is_linkonce(sname)) { - if (!this->include_linkonce_section(layout, i, name, shdr)) + if (!this->include_linkonce_section(alayout, i, sname, shdr)) discard = true; } } @@ -1174,7 +1174,7 @@ Sized_relobj::do_layout(Symbol_table* symtab, if (is_gc_pass_one && parameters->options().gc_sections()) { - if (is_section_name_included(name) + if (is_section_name_included(sname) || shdr.get_sh_type() == elfcpp::SHT_INIT_ARRAY || shdr.get_sh_type() == elfcpp::SHT_FINI_ARRAY) { @@ -1188,7 +1188,7 @@ Sized_relobj::do_layout(Symbol_table* symtab, // However, we don't know that yet for all sections. So save // reloc sections and process them later. Garbage collection is // not triggered when relocatable code is desired. - if (emit_relocs + if (emit_rels && (shdr.get_sh_type() == elfcpp::SHT_REL || shdr.get_sh_type() == elfcpp::SHT_RELA)) { @@ -1206,7 +1206,7 @@ Sized_relobj::do_layout(Symbol_table* symtab, // determine which sections are being discarded, and discard the // corresponding information. if (!relocatable - && strcmp(name, ".eh_frame") == 0 + && strcmp(sname, ".eh_frame") == 0 && this->check_eh_frame_flags(&shdr)) { if (is_gc_pass_one) @@ -1279,7 +1279,7 @@ Sized_relobj::do_layout(Symbol_table* symtab, if (should_defer_layout && (shdr.get_sh_flags() & elfcpp::SHF_ALLOC)) { gold_assert(!is_gc_pass_two); - this->deferred_layout_.push_back(Deferred_layout(i, name, + this->deferred_layout_.push_back(Deferred_layout(i, sname, pshdrs, reloc_shndx[i], reloc_type[i])); @@ -1308,18 +1308,18 @@ Sized_relobj::do_layout(Symbol_table* symtab, { // When garbage collection is switched on the actual layout // only happens in the second call. - this->layout_section(layout, i, name, shdr, reloc_shndx[i], + this->layout_section(alayout, i, sname, shdr, reloc_shndx[i], reloc_type[i]); } } if (!is_gc_pass_one) - layout->layout_gnu_stack(seen_gnu_stack, gnu_stack_flags); + alayout->layout_gnu_stack(seen_gnu_stack, gnu_stack_flags); // When doing a relocatable link handle the reloc sections at the // end. Garbage collection and Identical Code Folding is not // turned on for relocatable code. - if (emit_relocs) + if (emit_rels) this->size_relocatable_relocs(); gold_assert(!(is_gc_or_icf) || reloc_sections.empty()); @@ -1334,7 +1334,7 @@ Sized_relobj::do_layout(Symbol_table* symtab, typename This::Shdr shdr(pshdr); unsigned int data_shndx = this->adjust_shndx(shdr.get_sh_info()); - if (data_shndx >= shnum) + if (data_shndx >= sec_shnum) { // We already warned about this above. continue; @@ -1351,8 +1351,8 @@ Sized_relobj::do_layout(Symbol_table* symtab, Relocatable_relocs* rr = new Relocatable_relocs(); this->set_relocatable_relocs(i, rr); - Output_section* os = layout->layout_reloc(this, i, shdr, data_section, - rr); + Output_section* os = alayout->layout_reloc(this, i, shdr, data_section, + rr); out_sections[i] = os; out_section_offsets[i] = invalid_address; } @@ -1371,18 +1371,18 @@ Sized_relobj::do_layout(Symbol_table* symtab, pshdr = section_headers_data + i * This::shdr_size; typename This::Shdr shdr(pshdr); - off_t offset; - Output_section* os = layout->layout_eh_frame(this, - symbols_data, - symbols_size, - symbol_names_data, - symbol_names_size, - i, shdr, - reloc_shndx[i], - reloc_type[i], - &offset); + off_t off; + Output_section* os = alayout->layout_eh_frame(this, + symbols_data, + symbols_size, + symbol_names_data, + symbol_names_size, + i, shdr, + reloc_shndx[i], + reloc_type[i], + &off); out_sections[i] = os; - if (offset == -1) + if (off == -1) { // An object can contain at most one section holding exception // frame information. @@ -1391,12 +1391,12 @@ Sized_relobj::do_layout(Symbol_table* symtab, out_section_offsets[i] = invalid_address; } else - out_section_offsets[i] = convert_types(offset); + out_section_offsets[i] = convert_types(off); // If this section requires special handling, and if there are // relocs that apply to it, then we must do the special handling // before we apply the relocs. - if (offset == -1 && reloc_shndx[i] != 0) + if (off == -1 && reloc_shndx[i] != 0) this->set_relocs_must_follow_section_writes(); } @@ -1422,7 +1422,7 @@ Sized_relobj::do_layout(Symbol_table* symtab, template void -Sized_relobj::do_layout_deferred_sections(Layout* layout) +Sized_relobj::do_layout_deferred_sections(Layout* alayout) { typename std::vector::iterator deferred; @@ -1431,7 +1431,7 @@ Sized_relobj::do_layout_deferred_sections(Layout* layout) ++deferred) { typename This::Shdr shdr(deferred->shdr_data_); - this->layout_section(layout, deferred->shndx_, deferred->name_.c_str(), + this->layout_section(alayout, deferred->shndx_, deferred->name_.c_str(), shdr, deferred->reloc_shndx_, deferred->reloc_type_); } @@ -1452,10 +1452,10 @@ Sized_relobj::do_add_symbols(Symbol_table* symtab, return; } - const int sym_size = This::sym_size; + const int symsize = This::sym_size; size_t symcount = ((sd->symbols_size - sd->external_symbols_offset) - / sym_size); - if (symcount * sym_size != sd->symbols_size - sd->external_symbols_offset) + / symsize); + if (symcount * symsize != sd->symbols_size - sd->external_symbols_offset) { this->error(_("size of symbols is not multiple of symbol size")); return; @@ -1497,16 +1497,16 @@ Sized_relobj::do_count_local_symbols(Stringpool* pool, } // Read the symbol table section header. - const unsigned int symtab_shndx = this->symtab_shndx_; + const unsigned int sym_tab_shndx = this->symtab_shndx_; typename This::Shdr symtabshdr(this, - this->elf_file_.section_header(symtab_shndx)); + this->elf_file_.section_header(sym_tab_shndx)); gold_assert(symtabshdr.get_sh_type() == elfcpp::SHT_SYMTAB); // Read the local symbols. - const int sym_size = This::sym_size; + const int symsize = This::sym_size; const unsigned int loccount = this->local_symbol_count_; gold_assert(loccount == symtabshdr.get_sh_info()); - off_t locsize = loccount * sym_size; + off_t locsize = loccount * symsize; const unsigned char* psyms = this->get_view(symtabshdr.get_sh_offset(), locsize, true, true); @@ -1522,13 +1522,13 @@ Sized_relobj::do_count_local_symbols(Stringpool* pool, // Loop over the local symbols. const Output_sections& out_sections(this->output_sections()); - unsigned int shnum = this->shnum(); + unsigned int sec_shnum = this->shnum(); unsigned int count = 0; unsigned int dyncount = 0; // Skip the first, dummy, symbol. - psyms += sym_size; + psyms += symsize; bool discard_locals = parameters->options().discard_locals(); - for (unsigned int i = 1; i < loccount; ++i, psyms += sym_size) + for (unsigned int i = 1; i < loccount; ++i, psyms += symsize) { elfcpp::Sym sym(psyms); @@ -1549,7 +1549,7 @@ Sized_relobj::do_count_local_symbols(Stringpool* pool, // Decide whether this symbol should go into the output file. - if ((shndx < shnum && out_sections[shndx] == NULL) + if ((shndx < sec_shnum && out_sections[shndx] == NULL) || (shndx == this->discarded_eh_frame_shndx_)) { lv.set_no_output_symtab_entry(); @@ -1585,11 +1585,11 @@ Sized_relobj::do_count_local_symbols(Stringpool* pool, // - the symbol has a name. // // We do not discard a symbol if it needs a dynamic symbol entry. - const char* name = pnames + sym.get_st_name(); + const char* sname = pnames + sym.get_st_name(); if (discard_locals && sym.get_st_type() != elfcpp::STT_FILE && !lv.needs_output_dynsym_entry() - && parameters->target().is_local_label_name(name)) + && parameters->target().is_local_label_name(sname)) { lv.set_no_output_symtab_entry(); continue; @@ -1597,20 +1597,20 @@ Sized_relobj::do_count_local_symbols(Stringpool* pool, // Discard the local symbol if -retain_symbols_file is specified // and the local symbol is not in that file. - if (!parameters->options().should_retain_symbol(name)) + if (!parameters->options().should_retain_symbol(sname)) { lv.set_no_output_symtab_entry(); continue; } // Add the symbol to the symbol table string pool. - pool->add(name, true, NULL); + pool->add(sname, true, NULL); ++count; // If needed, add the symbol to the dynamic symbol table string pool. if (lv.needs_output_dynsym_entry()) { - dynpool->add(name, true, NULL); + dynpool->add(sname, true, NULL); ++dyncount; } } @@ -1638,7 +1638,7 @@ Sized_relobj::do_finalize_local_symbols(unsigned int index, const bool relocatable = parameters->options().relocatable(); const Output_sections& out_sections(this->output_sections()); const std::vector
& out_offsets(this->section_offsets_); - unsigned int shnum = this->shnum(); + unsigned int sec_shnum = this->shnum(); for (unsigned int i = 1; i < loccount; ++i) { @@ -1662,7 +1662,7 @@ Sized_relobj::do_finalize_local_symbols(unsigned int index, } else { - if (shndx >= shnum) + if (shndx >= sec_shnum) { this->error(_("local symbol %u section index %u out of range"), i, shndx); @@ -1854,16 +1854,16 @@ Sized_relobj::write_local_symbols( } // Read the symbol table section header. - const unsigned int symtab_shndx = this->symtab_shndx_; + const unsigned int sym_tab_shndx = this->symtab_shndx_; typename This::Shdr symtabshdr(this, - this->elf_file_.section_header(symtab_shndx)); + this->elf_file_.section_header(sym_tab_shndx)); gold_assert(symtabshdr.get_sh_type() == elfcpp::SHT_SYMTAB); const unsigned int loccount = this->local_symbol_count_; gold_assert(loccount == symtabshdr.get_sh_info()); // Read the local symbols. - const int sym_size = This::sym_size; - off_t locsize = loccount * sym_size; + const int symsize = This::sym_size; + off_t locsize = loccount * symsize; const unsigned char* psyms = this->get_view(symtabshdr.get_sh_offset(), locsize, true, false); @@ -1878,12 +1878,12 @@ Sized_relobj::write_local_symbols( // Get views into the output file for the portions of the symbol table // and the dynamic symbol table that we will be writing. - off_t output_size = this->output_local_symbol_count_ * sym_size; + off_t output_size = this->output_local_symbol_count_ * symsize; unsigned char* oview = NULL; if (output_size > 0) oview = of->get_output_view(this->local_symbol_offset_, output_size); - off_t dyn_output_size = this->output_local_dynsym_count_ * sym_size; + off_t dyn_output_size = this->output_local_dynsym_count_ * symsize; unsigned char* dyn_oview = NULL; if (dyn_output_size > 0) dyn_oview = of->get_output_view(this->local_dynsym_offset_, @@ -1895,8 +1895,8 @@ Sized_relobj::write_local_symbols( unsigned char* ov = oview; unsigned char* dyn_ov = dyn_oview; - psyms += sym_size; - for (unsigned int i = 1; i < loccount; ++i, psyms += sym_size) + psyms += symsize; + for (unsigned int i = 1; i < loccount; ++i, psyms += symsize) { elfcpp::Sym isym(psyms); @@ -1927,15 +1927,15 @@ Sized_relobj::write_local_symbols( elfcpp::Sym_write osym(ov); gold_assert(isym.get_st_name() < strtab_size); - const char* name = pnames + isym.get_st_name(); - osym.put_st_name(sympool->get_offset(name)); + const char* sname = pnames + isym.get_st_name(); + osym.put_st_name(sympool->get_offset(sname)); osym.put_st_value(this->local_values_[i].value(this, 0)); osym.put_st_size(isym.get_st_size()); osym.put_st_info(isym.get_st_info()); osym.put_st_other(isym.get_st_other()); osym.put_st_shndx(st_shndx); - ov += sym_size; + ov += symsize; } // Write the symbol to the output dynamic symbol table. @@ -1945,15 +1945,15 @@ Sized_relobj::write_local_symbols( elfcpp::Sym_write osym(dyn_ov); gold_assert(isym.get_st_name() < strtab_size); - const char* name = pnames + isym.get_st_name(); - osym.put_st_name(dynpool->get_offset(name)); + const char* sname = pnames + isym.get_st_name(); + osym.put_st_name(dynpool->get_offset(sname)); osym.put_st_value(this->local_values_[i].value(this, 0)); osym.put_st_size(isym.get_st_size()); osym.put_st_info(isym.get_st_info()); osym.put_st_other(isym.get_st_other()); osym.put_st_shndx(st_shndx); - dyn_ov += sym_size; + dyn_ov += symsize; } } @@ -1980,7 +1980,7 @@ template bool Sized_relobj::get_symbol_location_info( unsigned int shndx, - off_t offset, + off_t sym_offset, Symbol_location_info* info) { if (this->symtab_shndx_ == 0) @@ -1998,11 +1998,11 @@ Sized_relobj::get_symbol_location_info( this->section_contents(symbol_names_shndx, &names_size, false); const char* symbol_names = reinterpret_cast(symbol_names_u); - const int sym_size = This::sym_size; - const size_t count = symbols_size / sym_size; + const int symsize = This::sym_size; + const size_t count = symbols_size / symsize; const unsigned char* p = symbols; - for (size_t i = 0; i < count; ++i, p += sym_size) + for (size_t i = 0; i < count; ++i, p += symsize) { elfcpp::Sym sym(p); @@ -2020,9 +2020,9 @@ Sized_relobj::get_symbol_location_info( &is_ordinary); if (is_ordinary && st_shndx == shndx - && static_cast(sym.get_st_value()) <= offset + && static_cast(sym.get_st_value()) <= sym_offset && (static_cast(sym.get_st_value() + sym.get_st_size()) - > offset)) + > sym_offset)) { if (sym.get_st_name() > names_size) info->enclosing_symbol_name = "(invalid)"; @@ -2065,11 +2065,11 @@ Sized_relobj::map_to_kept_section( Sized_relobj* kept_relobj = static_cast*>(kept_object); Output_section* os = kept_relobj->output_section(kept_shndx); - Address offset = kept_relobj->get_output_section_offset(kept_shndx); - if (os != NULL && offset != invalid_address) + Address addr_offset = kept_relobj->get_output_section_offset(kept_shndx); + if (os != NULL && addr_offset != invalid_address) { *found = true; - return os->address() + offset; + return os->address() + addr_offset; } } *found = false; -- cgit v1.1