diff options
author | Roger Kim <rgr@fb.com> | 2021-11-08 10:26:50 -0800 |
---|---|---|
committer | Shoaib Meenai <smeenai@fb.com> | 2021-11-08 10:33:59 -0800 |
commit | 1658980a1c82531774d04632fc4dfec9a3ed0f82 (patch) | |
tree | 6f12f66e911e3e3b2ca840469dcc23f6c9365801 /llvm/tools/llvm-libtool-darwin/llvm-libtool-darwin.cpp | |
parent | c51f947a1351509dd68d8fe5843794ce0201b937 (diff) | |
download | llvm-1658980a1c82531774d04632fc4dfec9a3ed0f82.zip llvm-1658980a1c82531774d04632fc4dfec9a3ed0f82.tar.gz llvm-1658980a1c82531774d04632fc4dfec9a3ed0f82.tar.bz2 |
[NFC][llvm-libtool-darwin] Clean up names
Removing unclear abbreviations.
Reviewed By: jhenderson
Differential Revision: https://reviews.llvm.org/D113215
Diffstat (limited to 'llvm/tools/llvm-libtool-darwin/llvm-libtool-darwin.cpp')
-rw-r--r-- | llvm/tools/llvm-libtool-darwin/llvm-libtool-darwin.cpp | 45 |
1 files changed, 24 insertions, 21 deletions
diff --git a/llvm/tools/llvm-libtool-darwin/llvm-libtool-darwin.cpp b/llvm/tools/llvm-libtool-darwin/llvm-libtool-darwin.cpp index 77bd042..fc7a35f 100644 --- a/llvm/tools/llvm-libtool-darwin/llvm-libtool-darwin.cpp +++ b/llvm/tools/llvm-libtool-darwin/llvm-libtool-darwin.cpp @@ -299,17 +299,18 @@ static Error verifyAndAddIRObject(MembersPerArchitectureMap &Members, static Error addChildMember(MembersPerArchitectureMap &Members, const object::Archive::Child &M, const Config &C) { - Expected<NewArchiveMember> NMOrErr = + Expected<NewArchiveMember> NewMemberOrErr = NewArchiveMember::getOldMember(M, C.Deterministic); - if (!NMOrErr) - return NMOrErr.takeError(); + if (!NewMemberOrErr) + return NewMemberOrErr.takeError(); + auto &NewMember = *NewMemberOrErr; - file_magic Magic = identify_magic(NMOrErr->Buf->getBuffer()); + file_magic Magic = identify_magic(NewMember.Buf->getBuffer()); if (Magic == file_magic::bitcode) - return verifyAndAddIRObject(Members, std::move(*NMOrErr), C); + return verifyAndAddIRObject(Members, std::move(NewMember), C); - return verifyAndAddMachOObject(Members, std::move(*NMOrErr), C); + return verifyAndAddMachOObject(Members, std::move(NewMember), C); } static Error processArchive(MembersPerArchitectureMap &Members, @@ -328,9 +329,10 @@ static Error processArchive(MembersPerArchitectureMap &Members, static Error addArchiveMembers(MembersPerArchitectureMap &Members, std::vector<std::unique_ptr<MemoryBuffer>> &ArchiveBuffers, - NewArchiveMember NM, StringRef FileName, const Config &C) { + NewArchiveMember NewMember, StringRef FileName, + const Config &C) { Expected<std::unique_ptr<Archive>> LibOrErr = - object::Archive::create(NM.Buf->getMemBufferRef()); + object::Archive::create(NewMember.Buf->getMemBufferRef()); if (!LibOrErr) return createFileError(FileName, LibOrErr.takeError()); @@ -339,16 +341,16 @@ addArchiveMembers(MembersPerArchitectureMap &Members, // Update vector ArchiveBuffers with the MemoryBuffers to transfer // ownership. - ArchiveBuffers.push_back(std::move(NM.Buf)); + ArchiveBuffers.push_back(std::move(NewMember.Buf)); return Error::success(); } static Error addUniversalMembers( MembersPerArchitectureMap &Members, std::vector<std::unique_ptr<MemoryBuffer>> &UniversalBuffers, - NewArchiveMember NM, StringRef FileName, const Config &C) { + NewArchiveMember NewMember, StringRef FileName, const Config &C) { Expected<std::unique_ptr<MachOUniversalBinary>> BinaryOrErr = - MachOUniversalBinary::create(NM.Buf->getMemBufferRef()); + MachOUniversalBinary::create(NewMember.Buf->getMemBufferRef()); if (!BinaryOrErr) return createFileError(FileName, BinaryOrErr.takeError()); @@ -407,38 +409,39 @@ static Error addUniversalMembers( // Update vector UniversalBuffers with the MemoryBuffers to transfer // ownership. - UniversalBuffers.push_back(std::move(NM.Buf)); + UniversalBuffers.push_back(std::move(NewMember.Buf)); return Error::success(); } static Error addMember(MembersPerArchitectureMap &Members, std::vector<std::unique_ptr<MemoryBuffer>> &FileBuffers, StringRef FileName, const Config &C) { - Expected<NewArchiveMember> NMOrErr = + Expected<NewArchiveMember> NewMemberOrErr = NewArchiveMember::getFile(FileName, C.Deterministic); - if (!NMOrErr) - return createFileError(FileName, NMOrErr.takeError()); + if (!NewMemberOrErr) + return createFileError(FileName, NewMemberOrErr.takeError()); + auto &NewMember = *NewMemberOrErr; // For regular archives, use the basename of the object path for the member // name. - NMOrErr->MemberName = sys::path::filename(NMOrErr->MemberName); - file_magic Magic = identify_magic(NMOrErr->Buf->getBuffer()); + NewMember.MemberName = sys::path::filename(NewMember.MemberName); + file_magic Magic = identify_magic(NewMember.Buf->getBuffer()); // Flatten archives. if (Magic == file_magic::archive) - return addArchiveMembers(Members, FileBuffers, std::move(*NMOrErr), + return addArchiveMembers(Members, FileBuffers, std::move(NewMember), FileName, C); // Flatten universal files. if (Magic == file_magic::macho_universal_binary) - return addUniversalMembers(Members, FileBuffers, std::move(*NMOrErr), + return addUniversalMembers(Members, FileBuffers, std::move(NewMember), FileName, C); // Bitcode files. if (Magic == file_magic::bitcode) - return verifyAndAddIRObject(Members, std::move(*NMOrErr), C); + return verifyAndAddIRObject(Members, std::move(NewMember), C); - return verifyAndAddMachOObject(Members, std::move(*NMOrErr), C); + return verifyAndAddMachOObject(Members, std::move(NewMember), C); } static Expected<SmallVector<Slice, 2>> |