[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