================
@@ -758,9 +758,14 @@ void darwin::Linker::ConstructJob(Compilation &C, const 
JobAction &JA,
     }
   }
 
-  // Add non-standard, platform-specific search paths, e.g., for DriverKit:
-  //  -L<sysroot>/System/DriverKit/usr/lib
-  //  -F<sysroot>/System/DriverKit/System/Library/Framework
+  // Add framework include paths and library search paths.
+  // There are two flavors:
+  // 1. The "non-standard" paths, e.g. for DriverKit:
+  //      -L<sysroot>/System/DriverKit/usr/lib
+  //      -F<sysroot>/System/DriverKit/System/Library/Frameworks
+  // 2. The "standard" paths, e.g. for macOS and iOS:
+  //      -F<sysroot>/System/Library/Frameworks
+  //      -F<sysroot>/Library/Frameworks
   {
     bool NonStandardSearchPath = false;
----------------
ldionne wrote:

@yln Note that I think this whole block could almost go away now. I think we 
always do the right thing in the linker on Apple platforms, so I don't think 
the driver needs any special logic for `-F` and `-L` anymore, but I could be 
wrong. I also wanted to keep this change minimal.

https://github.com/llvm/llvm-project/pull/75841
_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Reply via email to