Support 64-bit entry size in SHT_HASH (for s390).

gold/
	* dynobj.cc (Dynobj::create_elf_hash_table): Create hash table with
	target-specific entry size.
	(Dynobj::sized_create_elf_hash_table): Add size template parameter.
	* dynobj.h (Dynobj::sized_create_elf_hash_table): Likewise.
	* layout.cc (Layout::create_dynamic_symtab): Set entsize to
	hash_entry_size.
	* target.h (Target::hash_entry_size): New method.
	(Target::Target_info::hash_entry_size): New data member.

	* aarch64.cc (Target_aarch64::aarch64_info): Add hash_entry_size.
	* arm.cc (Target_arm::arm_info): Likewise.
	(Target_arm_nacl::arm_nacl_info): Likewise.
	* i386.cc (Target_i386::i386_info): Likewise.
	(Target_i386_nacl::i386_nacl_info): Likewise.
	(Target_iamcu::iamcu_info): Likewise.
	* mips.cc (Target_mips::mips_info): Likewise.
	(Target_mips_nacl::mips_nacl_info): Likewise.
	* powerpc.cc (Target_powerpc::powerpc_info): Likewise.
	* sparc.cc (Target_sparc::sparc_info): Likewise.
	* tilegx.cc (Target_tilegx::tilegx_info): Likewise.
	* x86_64.cc (Target_x86_64::x86_64_info): Likewise.
	(Target_x86_64_nacl::x86_64_nacl_info): Likewise.
	* testsuite/testfile.cc (Target_test::test_target_info): Likewise.
This commit is contained in:
Marcin Kościelnicki 2015-10-05 16:57:11 +02:00 committed by Cary Coutant
parent 93084fcd9b
commit 8d9743bd43
14 changed files with 138 additions and 50 deletions

View File

@ -1,3 +1,29 @@
2015-10-28 Marcin Kościelnicki <koriakin@0x04.net>
* dynobj.cc (Dynobj::create_elf_hash_table): Create hash table with
target-specific entry size.
(Dynobj::sized_create_elf_hash_table): Add size template parameter.
* dynobj.h (Dynobj::sized_create_elf_hash_table): Likewise.
* layout.cc (Layout::create_dynamic_symtab): Set entsize to
hash_entry_size.
* target.h (Target::hash_entry_size): New method.
(Target::Target_info::hash_entry_size): New data member.
* aarch64.cc (Target_aarch64::aarch64_info): Add hash_entry_size.
* arm.cc (Target_arm::arm_info): Likewise.
(Target_arm_nacl::arm_nacl_info): Likewise.
* i386.cc (Target_i386::i386_info): Likewise.
(Target_i386_nacl::i386_nacl_info): Likewise.
(Target_iamcu::iamcu_info): Likewise.
* mips.cc (Target_mips::mips_info): Likewise.
(Target_mips_nacl::mips_nacl_info): Likewise.
* powerpc.cc (Target_powerpc::powerpc_info): Likewise.
* sparc.cc (Target_sparc::sparc_info): Likewise.
* tilegx.cc (Target_tilegx::tilegx_info): Likewise.
* x86_64.cc (Target_x86_64::x86_64_info): Likewise.
(Target_x86_64_nacl::x86_64_nacl_info): Likewise.
* testsuite/testfile.cc (Target_test::test_target_info): Likewise.
2015-10-28 H.J. Lu <hongjiu.lu@intel.com>
PR gold/19177

View File

@ -3434,7 +3434,8 @@ const Target::Target_info Target_aarch64<64, false>::aarch64_info =
0, // large_common_section_flags
NULL, // attributes_section
NULL, // attributes_vendor
"_start" // entry_symbol_name
"_start", // entry_symbol_name
32, // hash_entry_size
};
template<>
@ -3461,7 +3462,8 @@ const Target::Target_info Target_aarch64<32, false>::aarch64_info =
0, // large_common_section_flags
NULL, // attributes_section
NULL, // attributes_vendor
"_start" // entry_symbol_name
"_start", // entry_symbol_name
32, // hash_entry_size
};
template<>
@ -3488,7 +3490,8 @@ const Target::Target_info Target_aarch64<64, true>::aarch64_info =
0, // large_common_section_flags
NULL, // attributes_section
NULL, // attributes_vendor
"_start" // entry_symbol_name
"_start", // entry_symbol_name
32, // hash_entry_size
};
template<>
@ -3515,7 +3518,8 @@ const Target::Target_info Target_aarch64<32, true>::aarch64_info =
0, // large_common_section_flags
NULL, // attributes_section
NULL, // attributes_vendor
"_start" // entry_symbol_name
"_start", // entry_symbol_name
32, // hash_entry_size
};
// Get the GOT section, creating it if necessary.

View File

@ -2997,7 +2997,8 @@ const Target::Target_info Target_arm<big_endian>::arm_info =
0, // large_common_section_flags
".ARM.attributes", // attributes_section
"aeabi", // attributes_vendor
"_start" // entry_symbol_name
"_start", // entry_symbol_name
32, // hash_entry_size
};
// Arm relocate functions class
@ -12744,7 +12745,8 @@ const Target::Target_info Target_arm_nacl<big_endian>::arm_nacl_info =
0, // large_common_section_flags
".ARM.attributes", // attributes_section
"aeabi", // attributes_vendor
"_start" // entry_symbol_name
"_start", // entry_symbol_name
32, // hash_entry_size
};
template<bool big_endian>

View File

@ -946,31 +946,59 @@ Dynobj::create_elf_hash_table(const std::vector<Symbol*>& dynsyms,
bucket[bucketpos] = dynsym_index;
}
int size = parameters->target().hash_entry_size();
unsigned int hashlen = ((2
+ bucketcount
+ local_dynsym_count
+ dynsym_count)
* 4);
* size / 8);
unsigned char* phash = new unsigned char[hashlen];
if (parameters->target().is_big_endian())
bool big_endian = parameters->target().is_big_endian();
if (size == 32)
{
if (big_endian)
{
#if defined(HAVE_TARGET_32_BIG) || defined(HAVE_TARGET_64_BIG)
Dynobj::sized_create_elf_hash_table<true>(bucket, chain, phash,
hashlen);
Dynobj::sized_create_elf_hash_table<32, true>(bucket, chain, phash,
hashlen);
#else
gold_unreachable();
gold_unreachable();
#endif
}
else
{
#if defined(HAVE_TARGET_32_LITTLE) || defined(HAVE_TARGET_64_LITTLE)
Dynobj::sized_create_elf_hash_table<32, false>(bucket, chain, phash,
hashlen);
#else
gold_unreachable();
#endif
}
}
else if (size == 64)
{
if (big_endian)
{
#if defined(HAVE_TARGET_32_BIG) || defined(HAVE_TARGET_64_BIG)
Dynobj::sized_create_elf_hash_table<64, true>(bucket, chain, phash,
hashlen);
#else
gold_unreachable();
#endif
}
else
{
#if defined(HAVE_TARGET_32_LITTLE) || defined(HAVE_TARGET_64_LITTLE)
Dynobj::sized_create_elf_hash_table<64, false>(bucket, chain, phash,
hashlen);
#else
gold_unreachable();
#endif
}
}
else
{
#if defined(HAVE_TARGET_32_LITTLE) || defined(HAVE_TARGET_64_LITTLE)
Dynobj::sized_create_elf_hash_table<false>(bucket, chain, phash,
hashlen);
#else
gold_unreachable();
#endif
}
gold_unreachable();
*pphash = phash;
*phashlen = hashlen;
@ -978,7 +1006,7 @@ Dynobj::create_elf_hash_table(const std::vector<Symbol*>& dynsyms,
// Fill in an ELF hash table.
template<bool big_endian>
template<int size, bool big_endian>
void
Dynobj::sized_create_elf_hash_table(const std::vector<uint32_t>& bucket,
const std::vector<uint32_t>& chain,
@ -990,21 +1018,21 @@ Dynobj::sized_create_elf_hash_table(const std::vector<uint32_t>& bucket,
const unsigned int bucketcount = bucket.size();
const unsigned int chaincount = chain.size();
elfcpp::Swap<32, big_endian>::writeval(p, bucketcount);
p += 4;
elfcpp::Swap<32, big_endian>::writeval(p, chaincount);
p += 4;
elfcpp::Swap<size, big_endian>::writeval(p, bucketcount);
p += size / 8;
elfcpp::Swap<size, big_endian>::writeval(p, chaincount);
p += size / 8;
for (unsigned int i = 0; i < bucketcount; ++i)
{
elfcpp::Swap<32, big_endian>::writeval(p, bucket[i]);
p += 4;
elfcpp::Swap<size, big_endian>::writeval(p, bucket[i]);
p += size / 8;
}
for (unsigned int i = 0; i < chaincount; ++i)
{
elfcpp::Swap<32, big_endian>::writeval(p, chain[i]);
p += 4;
elfcpp::Swap<size, big_endian>::writeval(p, chain[i]);
p += size / 8;
}
gold_assert(static_cast<unsigned int>(p - phash) == hashlen);

View File

@ -132,7 +132,7 @@ class Dynobj : public Object
bool for_gnu_hash_table);
// Sized version of create_elf_hash_table.
template<bool big_endian>
template<int size, bool big_endian>
static void
sized_create_elf_hash_table(const std::vector<uint32_t>& bucket,
const std::vector<uint32_t>& chain,

View File

@ -898,7 +898,8 @@ const Target::Target_info Target_i386::i386_info =
0, // large_common_section_flags
NULL, // attributes_section
NULL, // attributes_vendor
"_start" // entry_symbol_name
"_start", // entry_symbol_name
32, // hash_entry_size
};
// Get the GOT section, creating it if necessary.
@ -4084,7 +4085,8 @@ const Target::Target_info Target_i386_nacl::i386_nacl_info =
0, // large_common_section_flags
NULL, // attributes_section
NULL, // attributes_vendor
"_start" // entry_symbol_name
"_start", // entry_symbol_name
32, // hash_entry_size
};
#define NACLMASK 0xe0 // 32-byte alignment mask
@ -4320,7 +4322,8 @@ const Target::Target_info Target_iamcu::iamcu_info =
0, // large_common_section_flags
NULL, // attributes_section
NULL, // attributes_vendor
"_start" // entry_symbol_name
"_start", // entry_symbol_name
32, // hash_entry_size
};
class Target_selector_iamcu : public Target_selector

View File

@ -4409,7 +4409,7 @@ Layout::create_dynamic_symtab(const Input_objects* input_objects,
{
if (dynsym != NULL)
hashsec->set_link_section(dynsym);
hashsec->set_entsize(4);
hashsec->set_entsize(parameters->target().hash_entry_size() / 8);
}
if (odyn != NULL)

View File

@ -10490,7 +10490,8 @@ const Target::Target_info Target_mips<size, big_endian>::mips_info =
0, // large_common_section_flags
NULL, // attributes_section
NULL, // attributes_vendor
"__start" // entry_symbol_name
"__start", // entry_symbol_name
32, // hash_entry_size
};
template<int size, bool big_endian>
@ -10529,7 +10530,8 @@ const Target::Target_info Target_mips_nacl<size, big_endian>::mips_nacl_info =
0, // large_common_section_flags
NULL, // attributes_section
NULL, // attributes_vendor
"_start" // entry_symbol_name
"_start", // entry_symbol_name
32, // hash_entry_size
};
// Target selector for Mips. Note this is never instantiated directly.

View File

@ -1352,7 +1352,8 @@ Target::Target_info Target_powerpc<32, true>::powerpc_info =
0, // large_common_section_flags
NULL, // attributes_section
NULL, // attributes_vendor
"_start" // entry_symbol_name
"_start", // entry_symbol_name
32, // hash_entry_size
};
template<>
@ -1379,7 +1380,8 @@ Target::Target_info Target_powerpc<32, false>::powerpc_info =
0, // large_common_section_flags
NULL, // attributes_section
NULL, // attributes_vendor
"_start" // entry_symbol_name
"_start", // entry_symbol_name
32, // hash_entry_size
};
template<>
@ -1406,7 +1408,8 @@ Target::Target_info Target_powerpc<64, true>::powerpc_info =
0, // large_common_section_flags
NULL, // attributes_section
NULL, // attributes_vendor
"_start" // entry_symbol_name
"_start", // entry_symbol_name
32, // hash_entry_size
};
template<>
@ -1433,7 +1436,8 @@ Target::Target_info Target_powerpc<64, false>::powerpc_info =
0, // large_common_section_flags
NULL, // attributes_section
NULL, // attributes_vendor
"_start" // entry_symbol_name
"_start", // entry_symbol_name
32, // hash_entry_size
};
inline bool

View File

@ -482,7 +482,8 @@ Target::Target_info Target_sparc<32, true>::sparc_info =
0, // large_common_section_flags
NULL, // attributes_section
NULL, // attributes_vendor
"_start" // entry_symbol_name
"_start", // entry_symbol_name
32, // hash_entry_size
};
template<>
@ -509,7 +510,8 @@ Target::Target_info Target_sparc<64, true>::sparc_info =
0, // large_common_section_flags
NULL, // attributes_section
NULL, // attributes_vendor
"_start" // entry_symbol_name
"_start", // entry_symbol_name
32, // hash_entry_size
};
// We have to take care here, even when operating in little-endian

View File

@ -455,6 +455,11 @@ class Target
entry_symbol_name() const
{ return this->pti_->entry_symbol_name; }
// Return the size in bits of SHT_HASH entry.
int
hash_entry_size() const
{ return this->pti_->hash_entry_size; }
// Whether the target has a custom set_dynsym_indexes method.
bool
has_custom_set_dynsym_indexes() const
@ -540,6 +545,9 @@ class Target
const char* attributes_vendor;
// Name of the main entry point to the program.
const char* entry_symbol_name;
// Size (in bits) of SHT_HASH entry. Always equal to 32, except for
// 64-bit S/390.
const int hash_entry_size;
};
Target(const Target_info* pti)

View File

@ -108,7 +108,8 @@ const Target::Target_info Target_test<size, big_endian>::test_target_info =
0, // large_common_section_flags
NULL, // attributes_section
NULL, // attributes_vendor
"_start" // entry_symbol_name
"_start", // entry_symbol_name
32, // hash_entry_size
};
// The test targets.

View File

@ -680,7 +680,8 @@ const Target::Target_info Target_tilegx<64, false>::tilegx_info =
0, // large_common_section_flags
NULL, // attributes_section
NULL, // attributes_vendor
"_start" // entry_symbol_name
"_start", // entry_symbol_name
32, // hash_entry_size
};
template<>
@ -707,7 +708,8 @@ const Target::Target_info Target_tilegx<32, false>::tilegx_info =
0, // large_common_section_flags
NULL, // attributes_section
NULL, // attributes_vendor
"_start" // entry_symbol_name
"_start", // entry_symbol_name
32, // hash_entry_size
};
template<>
@ -734,7 +736,8 @@ const Target::Target_info Target_tilegx<64, true>::tilegx_info =
0, // large_common_section_flags
NULL, // attributes_section
NULL, // attributes_vendor
"_start" // entry_symbol_name
"_start", // entry_symbol_name
32, // hash_entry_size
};
template<>
@ -761,7 +764,8 @@ const Target::Target_info Target_tilegx<32, true>::tilegx_info =
0, // large_common_section_flags
NULL, // attributes_section
NULL, // attributes_vendor
"_start" // entry_symbol_name
"_start", // entry_symbol_name
32, // hash_entry_size
};
// tilegx relocation handlers

View File

@ -1052,7 +1052,8 @@ const Target::Target_info Target_x86_64<64>::x86_64_info =
elfcpp::SHF_X86_64_LARGE, // large_common_section_flags
NULL, // attributes_section
NULL, // attributes_vendor
"_start" // entry_symbol_name
"_start", // entry_symbol_name
32, // hash_entry_size
};
template<>
@ -1079,7 +1080,8 @@ const Target::Target_info Target_x86_64<32>::x86_64_info =
elfcpp::SHF_X86_64_LARGE, // large_common_section_flags
NULL, // attributes_section
NULL, // attributes_vendor
"_start" // entry_symbol_name
"_start", // entry_symbol_name
32, // hash_entry_size
};
// This is called when a new output section is created. This is where
@ -4818,7 +4820,8 @@ const Target::Target_info Target_x86_64_nacl<64>::x86_64_nacl_info =
elfcpp::SHF_X86_64_LARGE, // large_common_section_flags
NULL, // attributes_section
NULL, // attributes_vendor
"_start" // entry_symbol_name
"_start", // entry_symbol_name
32, // hash_entry_size
};
template<>
@ -4845,7 +4848,8 @@ const Target::Target_info Target_x86_64_nacl<32>::x86_64_nacl_info =
elfcpp::SHF_X86_64_LARGE, // large_common_section_flags
NULL, // attributes_section
NULL, // attributes_vendor
"_start" // entry_symbol_name
"_start", // entry_symbol_name
32, // hash_entry_size
};
#define NACLMASK 0xe0 // 32-byte alignment mask.