[clang] [clang][Darwin] Remove legacy framework search path logic in the frontend (PR #75841)

Louis Dionne via cfe-commits cfe-commits at lists.llvm.org
Mon Dec 18 10:45:21 PST 2023


================
@@ -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


More information about the cfe-commits mailing list