[llvm-objcopy][MachO] Minor refactoring of MachOLayoutBuilder

Move the implementation of getStringTableBuilderKind into
MachOLayoutBuilder.cpp. NFC.

Test plan: make check-all
This commit is contained in:
Alexander Shaposhnikov 2020-11-07 19:58:19 -08:00
parent 3ba3342232
commit 273ba73689
2 changed files with 9 additions and 6 deletions

View File

@ -15,6 +15,14 @@ namespace llvm {
namespace objcopy {
namespace macho {
StringTableBuilder::Kind
MachOLayoutBuilder::getStringTableBuilderKind(const Object &O, bool Is64Bit) {
if (O.Header.FileType == MachO::HeaderFileType::MH_OBJECT)
return Is64Bit ? StringTableBuilder::MachO64 : StringTableBuilder::MachO;
return Is64Bit ? StringTableBuilder::MachO64Linked
: StringTableBuilder::MachOLinked;
}
uint32_t MachOLayoutBuilder::computeSizeOfCmds() const {
uint32_t Size = 0;
for (const LoadCommand &LC : O.LoadCommands) {

View File

@ -34,12 +34,7 @@ class MachOLayoutBuilder {
Error layoutTail(uint64_t Offset);
static StringTableBuilder::Kind getStringTableBuilderKind(const Object &O,
bool Is64Bit) {
if (O.Header.FileType == MachO::HeaderFileType::MH_OBJECT)
return Is64Bit ? StringTableBuilder::MachO64 : StringTableBuilder::MachO;
return Is64Bit ? StringTableBuilder::MachO64Linked
: StringTableBuilder::MachOLinked;
}
bool Is64Bit);
public:
MachOLayoutBuilder(Object &O, bool Is64Bit, uint64_t PageSize)