[clang] [clang][modules] Determine if the SDK supports builtin modules independent of the target (PR #134005)

[Volodymyr Sapsai via cfe-commits]

================
@@ -2990,26 +2996,18 @@ static bool sdkSupportsBuiltinModules(
     return false;
 
   VersionTuple SDKVersion = SDKInfo->getVersion();
-  switch (TargetPlatform) {
+  switch (SDKInfo->getOS()) {
   // Existing SDKs added support for builtin modules in the fall
   // 2024 major releases.
-  case Darwin::MacOS:
+  case llvm::Triple::MacOSX:
     return SDKVersion >= VersionTuple(15U);
-  case Darwin::IPhoneOS:
-    switch (TargetEnvironment) {
-    case Darwin::MacCatalyst:
----------------
vsapsai wrote:

How is it handled in the new version? I don't see a special case for MacCatalyst in the new code, though I could have missed it.

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