[PATCH] D130137: [lld-macho] Simplify archive loading logic

[Nico Weber via Phabricator via llvm-commits]

thakis added inline comments.


================
Comment at: lld/MachO/Driver.cpp:299
       // using the cached archive should be enough
-      if (entry->second.isCommandLineLoad ||
-          entry->second.isCommandLineLoad == isCommandLineLoad)
+      if (entry->second.isCommandLineLoad || isCommandLineLoad == false)
         return file;
----------------
`!isCommandLineLoad` instead of `== false`


================
Comment at: lld/MachO/Driver.cpp:304
+    bool isLCLinkerForceLoad = loadType == LoadType::LCLinkerOption &&
+                               config->forceLoadSwift &&
+                               path::filename(path).startswith("libswift");
----------------
Why `&& forceLoadSwift && path::filename(path).startswith("libswift")`? Does this in LC_LINKER_OPTIONS only apply to swift archives?


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D130137/new/

https://reviews.llvm.org/D130137