[clang] macCatalyst: add SDKSettings.json as a dependency file if its potentially needed by the compiler (PR #177748)

Steven Wu via cfe-commits cfe-commits at lists.llvm.org
Mon Jan 26 09:14:23 PST 2026


================
@@ -3289,6 +3289,18 @@ void Darwin::addClangTargetOptions(
 
   addClangCC1ASTargetOptions(DriverArgs, CC1Args);
 
+  if (SDKInfo) {
+    // Make the SDKSettings.json an explicit dependency for the compiler
+    // invocation, in case the compiler needs to read it to remap macCatalyst
+    // versions.
+    if ((isTargetMacCatalyst() || TargetVariantTriple) &&
----------------
cachemeifyoucan wrote:

I am thinking we should just remove this constraint. We always use `SDKSettings` when available and we should always make it a dependency.

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


More information about the cfe-commits mailing list