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

[Cyndy Ishida via cfe-commits]

================
@@ -2539,6 +2550,18 @@ void DarwinClang::AddClangSystemIncludeArgs(const llvm::opt::ArgList &DriverArgs
     llvm::sys::path::append(P, "usr", "include");
     addExternCSystemInclude(DriverArgs, CC1Args, P.str());
   }
+
+  // Add default framework search paths
+  auto addFrameworkInclude = [&](auto ...Path) {
+    SmallString<128> P(Sysroot);
+    llvm::sys::path::append(P, Path...);
+
+    CC1Args.push_back("-internal-iframework");
+    CC1Args.push_back(DriverArgs.MakeArgString(P));
+  };
+  addFrameworkInclude("System", "Library", "Frameworks");
----------------
cyndyishida wrote:

Do you think it's possible to have the same string input style for linker search paths as header search paths? It would make sharing common the paths simpler, as a future cleanup.

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