[lld] [lld][MachO] Respect dylibs linked with `-allowable_client` (PR #114638)

Carlo Cabrera via llvm-commits llvm-commits at lists.llvm.org
Tue Nov 19 07:48:36 PST 2024


================
@@ -407,8 +407,28 @@ static InputFile *addFile(StringRef path, LoadType loadType,
   case file_magic::macho_dynamically_linked_shared_lib_stub:
   case file_magic::tapi_file:
     if (DylibFile *dylibFile =
-            loadDylib(mbref, nullptr, /*isBundleLoader=*/false, isExplicit))
+            loadDylib(mbref, nullptr, /*isBundleLoader=*/false, isExplicit)) {
+      if (isExplicit && !dylibFile->allowableClients.empty()) {
----------------
carlocab wrote:

I looked at that initially, but we would need to pass the client name to `loadDylib` or the constructor, and that seemed a bit much for me.

Would you rather I moved it there?

https://github.com/llvm/llvm-project/pull/114638


More information about the llvm-commits mailing list