[PATCH] D92360: [lld/mac] Fix issues around thin archives
Jez Ng via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Mon Nov 30 21:17:49 PST 2020
int3 added inline comments.
================
Comment at: lld/MachO/Driver.cpp:306
for (StringRef path : args::getLines(mbref))
- addFile(path);
-}
-
-static void forceLoadArchive(StringRef path) {
- if (Optional<MemoryBufferRef> buffer = readFile(path))
- for (MemoryBufferRef member : getArchiveMembers(*buffer))
- inputFiles.push_back(make<ObjFile>(member));
+ addFile(path, false);
}
----------------
int3 wrote:
> same for the rest of the calls
actually, I guess this would make for rather verbose code. How about making the parameter default to `false` instead?
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D92360/new/
https://reviews.llvm.org/D92360
More information about the llvm-commits
mailing list