Skip to content

Commit 1658980

Browse files
Roger Kimsmeenai
authored andcommitted
[NFC][llvm-libtool-darwin] Clean up names
Removing unclear abbreviations. Reviewed By: jhenderson Differential Revision: https://reviews.llvm.org/D113215
1 parent c51f947 commit 1658980

File tree

1 file changed

+24
-21
lines changed

1 file changed

+24
-21
lines changed

llvm/tools/llvm-libtool-darwin/llvm-libtool-darwin.cpp

Lines changed: 24 additions & 21 deletions
Original file line numberDiff line numberDiff line change
@@ -299,17 +299,18 @@ static Error verifyAndAddIRObject(MembersPerArchitectureMap &Members,
299299

300300
static Error addChildMember(MembersPerArchitectureMap &Members,
301301
const object::Archive::Child &M, const Config &C) {
302-
Expected<NewArchiveMember> NMOrErr =
302+
Expected<NewArchiveMember> NewMemberOrErr =
303303
NewArchiveMember::getOldMember(M, C.Deterministic);
304-
if (!NMOrErr)
305-
return NMOrErr.takeError();
304+
if (!NewMemberOrErr)
305+
return NewMemberOrErr.takeError();
306+
auto &NewMember = *NewMemberOrErr;
306307

307-
file_magic Magic = identify_magic(NMOrErr->Buf->getBuffer());
308+
file_magic Magic = identify_magic(NewMember.Buf->getBuffer());
308309

309310
if (Magic == file_magic::bitcode)
310-
return verifyAndAddIRObject(Members, std::move(*NMOrErr), C);
311+
return verifyAndAddIRObject(Members, std::move(NewMember), C);
311312

312-
return verifyAndAddMachOObject(Members, std::move(*NMOrErr), C);
313+
return verifyAndAddMachOObject(Members, std::move(NewMember), C);
313314
}
314315

315316
static Error processArchive(MembersPerArchitectureMap &Members,
@@ -328,9 +329,10 @@ static Error processArchive(MembersPerArchitectureMap &Members,
328329
static Error
329330
addArchiveMembers(MembersPerArchitectureMap &Members,
330331
std::vector<std::unique_ptr<MemoryBuffer>> &ArchiveBuffers,
331-
NewArchiveMember NM, StringRef FileName, const Config &C) {
332+
NewArchiveMember NewMember, StringRef FileName,
333+
const Config &C) {
332334
Expected<std::unique_ptr<Archive>> LibOrErr =
333-
object::Archive::create(NM.Buf->getMemBufferRef());
335+
object::Archive::create(NewMember.Buf->getMemBufferRef());
334336
if (!LibOrErr)
335337
return createFileError(FileName, LibOrErr.takeError());
336338

@@ -339,16 +341,16 @@ addArchiveMembers(MembersPerArchitectureMap &Members,
339341

340342
// Update vector ArchiveBuffers with the MemoryBuffers to transfer
341343
// ownership.
342-
ArchiveBuffers.push_back(std::move(NM.Buf));
344+
ArchiveBuffers.push_back(std::move(NewMember.Buf));
343345
return Error::success();
344346
}
345347

346348
static Error addUniversalMembers(
347349
MembersPerArchitectureMap &Members,
348350
std::vector<std::unique_ptr<MemoryBuffer>> &UniversalBuffers,
349-
NewArchiveMember NM, StringRef FileName, const Config &C) {
351+
NewArchiveMember NewMember, StringRef FileName, const Config &C) {
350352
Expected<std::unique_ptr<MachOUniversalBinary>> BinaryOrErr =
351-
MachOUniversalBinary::create(NM.Buf->getMemBufferRef());
353+
MachOUniversalBinary::create(NewMember.Buf->getMemBufferRef());
352354
if (!BinaryOrErr)
353355
return createFileError(FileName, BinaryOrErr.takeError());
354356

@@ -407,38 +409,39 @@ static Error addUniversalMembers(
407409

408410
// Update vector UniversalBuffers with the MemoryBuffers to transfer
409411
// ownership.
410-
UniversalBuffers.push_back(std::move(NM.Buf));
412+
UniversalBuffers.push_back(std::move(NewMember.Buf));
411413
return Error::success();
412414
}
413415

414416
static Error addMember(MembersPerArchitectureMap &Members,
415417
std::vector<std::unique_ptr<MemoryBuffer>> &FileBuffers,
416418
StringRef FileName, const Config &C) {
417-
Expected<NewArchiveMember> NMOrErr =
419+
Expected<NewArchiveMember> NewMemberOrErr =
418420
NewArchiveMember::getFile(FileName, C.Deterministic);
419-
if (!NMOrErr)
420-
return createFileError(FileName, NMOrErr.takeError());
421+
if (!NewMemberOrErr)
422+
return createFileError(FileName, NewMemberOrErr.takeError());
423+
auto &NewMember = *NewMemberOrErr;
421424

422425
// For regular archives, use the basename of the object path for the member
423426
// name.
424-
NMOrErr->MemberName = sys::path::filename(NMOrErr->MemberName);
425-
file_magic Magic = identify_magic(NMOrErr->Buf->getBuffer());
427+
NewMember.MemberName = sys::path::filename(NewMember.MemberName);
428+
file_magic Magic = identify_magic(NewMember.Buf->getBuffer());
426429

427430
// Flatten archives.
428431
if (Magic == file_magic::archive)
429-
return addArchiveMembers(Members, FileBuffers, std::move(*NMOrErr),
432+
return addArchiveMembers(Members, FileBuffers, std::move(NewMember),
430433
FileName, C);
431434

432435
// Flatten universal files.
433436
if (Magic == file_magic::macho_universal_binary)
434-
return addUniversalMembers(Members, FileBuffers, std::move(*NMOrErr),
437+
return addUniversalMembers(Members, FileBuffers, std::move(NewMember),
435438
FileName, C);
436439

437440
// Bitcode files.
438441
if (Magic == file_magic::bitcode)
439-
return verifyAndAddIRObject(Members, std::move(*NMOrErr), C);
442+
return verifyAndAddIRObject(Members, std::move(NewMember), C);
440443

441-
return verifyAndAddMachOObject(Members, std::move(*NMOrErr), C);
444+
return verifyAndAddMachOObject(Members, std::move(NewMember), C);
442445
}
443446

444447
static Expected<SmallVector<Slice, 2>>

0 commit comments

Comments
 (0)