[lld] [lld] handle re-exports for full framework paths (PR #137989)

Daniel Rodríguez Troitiño via llvm-commits llvm-commits at lists.llvm.org
Wed Apr 30 13:53:24 PDT 2025


================
@@ -1566,6 +1566,25 @@ static DylibFile *loadDylib(StringRef path, DylibFile *umbrella) {
   return loadDylib(*mbref, umbrella);
 }
 
+static StringRef findFramework(StringRef path, StringRef frameworkName) {
----------------
drodriguez wrote:

I think `findFramework` works because it is called from `findDylib` (which not only finds dylibs). This also should discard cases like embedded libraries (`One.framework/Resources/Library`) so being clear that we are looking for the framework and discarding other things should be clear in the function name. I like the extra comment, though.

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


More information about the llvm-commits mailing list