intel/llvm
1
0
Fork 0
mirror of https://github.com/intel/llvm.git synced 2026-01-23 07:58:23 +08:00
Code Issues Packages Projects Releases Wiki Activity

Replace duplicated '16' with a named constant.

Browse Source 
Thanks to Michael Spencer for the suggestion.

llvm-svn: 286462
This commit is contained in:
Rafael Espindola
2016-11-10 15:41:34 +00:00
parent ca57e53ded
commit 0e876cfb2e
2 changed files with 4 additions and 3 deletions
Download Patch File Download Diff File Expand all files Collapse all files

5
lld/ELF/SyntheticSections.cpp
View File

@@ -217,7 +217,7 @@ BuildIdSection<ELFT>::BuildIdSection(size_t HashSize)
HashSize(HashSize) {
this->Live = true;
Buf.resize(16 + HashSize);
Buf.resize(HeaderSize + HashSize);
const endianness E = ELFT::TargetEndianness;
write32<E>(Buf.data(), 4); // Name size
write32<E>(Buf.data() + 4, HashSize); // Content size
@@ -229,8 +229,7 @@ BuildIdSection<ELFT>::BuildIdSection(size_t HashSize)
// Returns the location of the build-id hash value in the output.
template <class ELFT>
uint8_t *BuildIdSection<ELFT>::getOutputLoc(uint8_t *Start) const {
// First 16 bytes are a header.
return Start + this->OutSec->Offset + this->OutSecOff + 16;
return Start + this->OutSec->Offset + this->OutSecOff + HeaderSize;
}
// Split one uint8 array into small pieces of uint8 arrays.

2
lld/ELF/SyntheticSections.h
View File

@@ -95,6 +95,8 @@ protected:
std::function<void(ArrayRef<uint8_t> Arr, uint8_t *Hash)> Hash);
size_t HashSize;
// First 16 bytes are a header.
static const unsigned HeaderSize = 16;
};
template <class ELFT>