[clang][driver][darwin] DarwinSDKInfo doesn't match sufficiently modified triples, doesn't warn for unsupported architectures (#204061)

DarwinSDKInfo::getPlatformPrefix(...) is passed the effective target
triple which has undergone modifications from the original -target
value, which can sometimes even include changes to the triple's
environment. That will cause it to fail to match the platform infos, not
get a platform prefix, and get incorrect default search paths. In the
case where a triple doesn't match, check all platform infos, and if they
all have the same platform prefix, then use that.

DarwinSDKInfo::supportsTriple(...) doesn't try to match the architecture
of the triple. That makes it not emit -Wincompatible-sysroot e.g. when
trying to use arm64 against a sufficiently old macOS SDK or when still
trying to build i386 for macOS. Instead of parsing values from
SDKSettings.json into a Triple, storing the probably-relevant
components, and then comparing those against a full Triple, just store
the parsed triple and get rid of the custom comparison code. While this
does make the matching problem above a little more fragile, in practice
that's well mitigated by treating thumb* and arm* as equivalent, and it
does allow for better diagnostics and overall simpler code.

Clean up some unnecessary copies in DarwinSDKInfo and related code.

Assisted-by: Claude Code

rdar://172876443
This commit is contained in:
Ian Anderson 2026-06-25 11:58:46 -07:00 committed by GitHub
parent 1813ca6e98
commit a916cb6b57
No known key found for this signature in database
GPG key ID: B5690EEEBB952194
7 changed files with 235 additions and 90 deletions

View file

@ -18,6 +18,7 @@
#include "llvm/Support/VersionTuple.h"
#include "llvm/Support/VirtualFileSystem.h"
#include "llvm/TargetParser/Triple.h"
#include <cassert>
#include <optional>
#include <string>
@ -36,32 +37,18 @@ public:
/// definitions, in the SDK.
struct SDKPlatformInfo {
public:
SDKPlatformInfo(llvm::Triple::VendorType Vendor, llvm::Triple::OSType OS,
llvm::Triple::EnvironmentType Environment,
llvm::Triple::ObjectFormatType ObjectFormat,
StringRef PlatformPrefix)
: Vendor(Vendor), OS(OS), Environment(Environment),
ObjectFormat(ObjectFormat), PlatformPrefix(PlatformPrefix) {}
using TripleStorageType = SmallVector<llvm::Triple, 5>;
llvm::Triple::VendorType getVendor() const { return Vendor; }
llvm::Triple::OSType getOS() const { return OS; }
llvm::Triple::EnvironmentType getEnvironment() const { return Environment; }
llvm::Triple::ObjectFormatType getObjectFormat() const {
return ObjectFormat;
SDKPlatformInfo(TripleStorageType Triples, StringRef PlatformPrefix)
: Triples(std::move(Triples)), PlatformPrefix(PlatformPrefix) {
assert(!this->Triples.empty() && "Triples cannot be empty");
}
const TripleStorageType &getTriples() const { return Triples; }
StringRef getPlatformPrefix() const { return PlatformPrefix; }
bool operator==(const llvm::Triple &RHS) const {
return (Vendor == RHS.getVendor()) && (OS == RHS.getOS()) &&
(Environment == RHS.getEnvironment()) &&
(ObjectFormat == RHS.getObjectFormat());
}
private:
llvm::Triple::VendorType Vendor;
llvm::Triple::OSType OS;
llvm::Triple::EnvironmentType Environment;
llvm::Triple::ObjectFormatType ObjectFormat;
TripleStorageType Triples;
std::string PlatformPrefix;
};
@ -190,11 +177,23 @@ public:
VersionMappings =
llvm::DenseMap<OSEnvPair::StorageType,
std::optional<RelatedTargetVersionMapping>>())
: FilePath(FilePath), OS(OS), Environment(Environment), Version(Version),
DisplayName(DisplayName),
: FilePath(std::move(FilePath)), OS(OS), Environment(Environment),
Version(Version), DisplayName(DisplayName),
MaximumDeploymentTarget(MaximumDeploymentTarget),
PlatformInfos(std::move(PlatformInfos)),
VersionMappings(std::move(VersionMappings)) {}
VersionMappings(std::move(VersionMappings)) {
assert(!this->PlatformInfos.empty() && "PlatformInfos cannot be empty");
}
/// Construct SDK Info inferred from the parameters rather than read from
/// SDKSettings.json.
///
/// This will infer \c PlatformInfos from an SDK for the OS/Environment that
/// predates the introduction of SDKSettings.json, and will not infer version
/// mappings.
DarwinSDKInfo(llvm::Triple::OSType OS,
llvm::Triple::EnvironmentType Environment, VersionTuple Version,
StringRef DisplayName, VersionTuple MaximumDeploymentTarget);
StringRef getFilePath() const { return FilePath; }
@ -206,20 +205,13 @@ public:
const StringRef getDisplayName() const { return DisplayName; }
const SDKPlatformInfo &getCanonicalPlatformInfo() const {
return PlatformInfos[0];
const llvm::Triple &getCanonicalPlatformTriple() const {
return PlatformInfos[0].getTriples()[0];
}
bool supportsTriple(llvm::Triple Triple) const {
return llvm::find(PlatformInfos, Triple) != PlatformInfos.end();
}
bool supportsTriple(const llvm::Triple &Triple) const;
const StringRef getPlatformPrefix(llvm::Triple Triple) const {
auto PlatformInfoIt = llvm::find(PlatformInfos, Triple);
if (PlatformInfoIt == PlatformInfos.end())
return StringRef();
return PlatformInfoIt->getPlatformPrefix();
}
StringRef getPlatformPrefix(const llvm::Triple &Triple) const;
// Returns the optional, target-specific version mapping that maps from one
// target to another target.

View file

@ -12,6 +12,7 @@
#include "llvm/Support/JSON.h"
#include "llvm/Support/MemoryBuffer.h"
#include "llvm/Support/Path.h"
#include "llvm/TargetParser/ARMTargetParser.h"
#include <optional>
using namespace clang;
@ -100,39 +101,118 @@ parseOSAndEnvironment(std::optional<StringRef> XcodePlatform) {
return {OS, Environment};
}
static DarwinSDKInfo::PlatformInfoStorageType
legacyPlatformInfos(llvm::Triple::OSType SDKOS,
llvm::Triple::EnvironmentType SDKEnvironment) {
DarwinSDKInfo::PlatformInfoStorageType PlatformInfos;
// Synthesize platform infos for older SDKs from the first SDKs with
// SupportedTargets: macOS 10.15 (DriverKit 19.0), iOS 13.0, tvOS 13.0,
// watchOS 6.0. Older macOS SDKs supported ppc and i386 architectures, and
// maybe ppc64. Support for those is difficult to identify from the SDK, and
// so aren't listed here. Other older SDKs (especially iOS) most likely
// supported armv6 and armv7 architectures that aren't listed here either.
switch (SDKOS) {
case llvm::Triple::MacOSX:
PlatformInfos.push_back({{llvm::Triple("x86_64-apple-macosx")}, ""});
// macOS 10.15 also has a Mac Catalyst supported target, but Mac Catalyst
// was new in that version so omit it for older versions.
break;
case llvm::Triple::DriverKit:
// DriverKit 19.0 only had SDKSettings.plist, which isn't used. So this code
// path is used in 19.x as well, not just earlier versions.
PlatformInfos.push_back(
{{llvm::Triple("x86_64-apple-driverkit")}, "/System/DriverKit"});
break;
case llvm::Triple::IOS:
switch (SDKEnvironment) {
case llvm::Triple::UnknownEnvironment:
PlatformInfos.push_back(
{{llvm::Triple("armv7-apple-ios"), llvm::Triple("armv7s-apple-ios"),
llvm::Triple("arm64-apple-ios")},
""});
break;
case llvm::Triple::Simulator:
PlatformInfos.push_back(
{{llvm::Triple("x86_64-apple-ios-simulator")}, ""});
break;
default:
break;
}
break;
case llvm::Triple::TvOS:
switch (SDKEnvironment) {
case llvm::Triple::UnknownEnvironment:
PlatformInfos.push_back({{llvm::Triple("arm64-apple-tvos")}, ""});
break;
case llvm::Triple::Simulator:
PlatformInfos.push_back(
{{llvm::Triple("x86_64-apple-tvos-simulator")}, ""});
break;
default:
break;
}
break;
case llvm::Triple::WatchOS:
switch (SDKEnvironment) {
case llvm::Triple::UnknownEnvironment:
PlatformInfos.push_back({{llvm::Triple("armv7k-apple-watchos"),
llvm::Triple("arm64_32-apple-watchos")},
""});
break;
case llvm::Triple::Simulator:
PlatformInfos.push_back(
{{llvm::Triple("x86_64-apple-watchos-simulator")}, ""});
break;
default:
break;
}
break;
case llvm::Triple::BridgeOS:
PlatformInfos.push_back({{llvm::Triple("armv7-apple-bridgeos"),
llvm::Triple("armv7s-apple-bridgeos"),
llvm::Triple("arm64-apple-bridgeos")},
""});
break;
default:
break;
}
return PlatformInfos;
}
static DarwinSDKInfo::PlatformInfoStorageType parsePlatformInfos(
const llvm::json::Object &Obj, std::optional<StringRef> XcodePlatform,
llvm::Triple::OSType SDKOS, llvm::Triple::EnvironmentType SDKEnvironment,
VersionTuple Version) {
DarwinSDKInfo::PlatformInfoStorageType PlatformInfos;
auto SupportedTargets = Obj.getObject("SupportedTargets");
if (!SupportedTargets) {
// For older SDKs that don't have SupportedTargets, infer one from the SDK's
// OS/Environment.
StringRef PlatformPrefix;
if (SDKOS == llvm::Triple::DriverKit)
PlatformPrefix = "/System/DriverKit";
PlatformInfos.push_back({llvm::Triple::Apple, SDKOS, SDKEnvironment,
llvm::Triple::MachO, PlatformPrefix});
return PlatformInfos;
}
if (!SupportedTargets)
return legacyPlatformInfos(SDKOS, SDKEnvironment);
for (auto SupportedTargetPair : *SupportedTargets) {
llvm::json::Object *SupportedTarget =
for (const auto &SupportedTargetPair : *SupportedTargets) {
const llvm::json::Object *SupportedTarget =
SupportedTargetPair.getSecond().getAsObject();
auto Vendor = SupportedTarget->getString("LLVMTargetTripleVendor");
auto OS = SupportedTarget->getString("LLVMTargetTripleSys");
if (!Vendor || !OS)
if (!SupportedTarget)
continue;
StringRef Arch = llvm::Triple::getArchName(llvm::Triple::UnknownArch);
auto Archs = SupportedTarget->getArray("Archs");
auto Vendor = SupportedTarget->getString("LLVMTargetTripleVendor");
auto OS = SupportedTarget->getString("LLVMTargetTripleSys");
if (!Archs || !Vendor || !OS)
continue;
DarwinSDKInfo::SDKPlatformInfo::TripleStorageType Triples;
auto Environment =
SupportedTarget->getString("LLVMTargetTripleEnvironment");
llvm::Triple Triple;
if (Environment)
Triple = llvm::Triple(Arch, *Vendor, *OS, *Environment);
else
Triple = llvm::Triple(Arch, *Vendor, *OS);
for (const auto &ArchValue : *Archs) {
if (auto Arch = ArchValue.getAsString()) {
if (Environment && !Environment->empty())
Triples.emplace_back(*Arch, *Vendor, *OS, *Environment);
else
Triples.emplace_back(*Arch, *Vendor, *OS);
}
}
if (Triples.empty())
continue;
// The key is either the Xcode platform, or a variant. The platform must be
// the first entry in the returned PlatformInfoStorageType.
@ -147,19 +227,17 @@ static DarwinSDKInfo::PlatformInfoStorageType parsePlatformInfos(
} else {
// Older SDKs don't have SystemPrefix in SupportedTargets, manually add
// their prefixes.
if ((Triple.getOS() == llvm::Triple::DriverKit) &&
if ((Triples[0].getOS() == llvm::Triple::DriverKit) &&
(Version < VersionTuple(22, 1)))
EffectivePlatformPrefix = "/System/DriverKit";
}
}
DarwinSDKInfo::SDKPlatformInfo PlatformInfo(
Triple.getVendor(), Triple.getOS(), Triple.getEnvironment(),
Triple.getObjectFormat(), EffectivePlatformPrefix);
if (PlatformOrVariant == XcodePlatform)
PlatformInfos.insert(PlatformInfos.begin(), PlatformInfo);
PlatformInfos.insert(PlatformInfos.begin(),
{std::move(Triples), EffectivePlatformPrefix});
else
PlatformInfos.push_back(PlatformInfo);
PlatformInfos.emplace_back(std::move(Triples), EffectivePlatformPrefix);
}
return PlatformInfos;
}
@ -195,6 +273,8 @@ DarwinSDKInfo::parseDarwinSDKSettingsJSON(std::string FilePath,
PlatformInfoStorageType PlatformInfos =
parsePlatformInfos(*Obj, XcodePlatform, OSAndEnvironment.first,
OSAndEnvironment.second, *Version);
if (PlatformInfos.empty())
return std::nullopt;
llvm::DenseMap<OSEnvPair::StorageType,
std::optional<RelatedTargetVersionMapping>>
VersionMappings;
@ -265,3 +345,72 @@ clang::parseDarwinSDKInfo(llvm::vfs::FileSystem &VFS, StringRef SDKRootPath) {
return llvm::make_error<llvm::StringError>("invalid SDKSettings.json",
llvm::inconvertibleErrorCode());
}
DarwinSDKInfo::DarwinSDKInfo(llvm::Triple::OSType OS,
llvm::Triple::EnvironmentType Environment,
VersionTuple Version, StringRef DisplayName,
VersionTuple MaximumDeploymentTarget)
: DarwinSDKInfo("", OS, Environment, Version, DisplayName,
MaximumDeploymentTarget,
legacyPlatformInfos(OS, Environment)) {}
static DarwinSDKInfo::PlatformInfoStorageType::const_iterator
findPlatformInfo(const DarwinSDKInfo::PlatformInfoStorageType &PlatformInfos,
const llvm::Triple &Triple) {
auto PlatformInfoIt = llvm::find_if(
PlatformInfos,
[&Triple](const DarwinSDKInfo::SDKPlatformInfo &PlatformInfo) {
const auto &Triples = PlatformInfo.getTriples();
return llvm::find(Triples, Triple) != Triples.end();
});
// The SDK specifies values for Xcode to use for the -target argument. It's
// hard to perfectly match the triple passed to this function against those
// values though. The passed in triple might have been computed from just
// -arch, or it might have been modified by -march and several other arguments
// that can effect any of the triple components. It's not really possible to
// account for all of the triple variations, but one common modification is
// that "arm" gets changed to "thumb". If the passed in triple is "thumb", try
// mapping it back to an "arm" triple since that's what the SDK will specify.
if (PlatformInfoIt == PlatformInfos.end() &&
Triple.getArch() == llvm::Triple::thumb) {
StringRef ARMArch = llvm::Triple::getArchName(llvm::Triple::arm);
// Preserve the sub-arch from the triple.
llvm::ARM::ArchKind ArchKind = llvm::ARM::parseArch(Triple.getArchName());
StringRef SubArch = llvm::ARM::getSubArch(ArchKind);
llvm::Triple ARMTriple(Triple);
ARMTriple.setArchName((ARMArch + SubArch).str());
if (ARMTriple.getArch() != llvm::Triple::thumb) {
// Sometimes changing the architecture name in the triple doesn't change
// its parsed architecture. e.g. armv6m parses as thumb, so it won't help
// to search for armv6m if thumbv6m already failed.
PlatformInfoIt = findPlatformInfo(PlatformInfos, ARMTriple);
}
}
return PlatformInfoIt;
}
bool DarwinSDKInfo::supportsTriple(const llvm::Triple &Triple) const {
return findPlatformInfo(PlatformInfos, Triple) != PlatformInfos.end();
}
StringRef DarwinSDKInfo::getPlatformPrefix(const llvm::Triple &Triple) const {
auto PlatformInfoIt = findPlatformInfo(PlatformInfos, Triple);
if (PlatformInfoIt != PlatformInfos.end())
return PlatformInfoIt->getPlatformPrefix();
// This triple probably isn't supported by the SDK. However, almost every SDK
// just has a single prefix where all of its contents are, so return that.
StringRef PlatformPrefix = PlatformInfos[0].getPlatformPrefix();
for (PlatformInfoIt = std::next(PlatformInfos.begin());
PlatformInfoIt != PlatformInfos.end(); ++PlatformInfoIt) {
if (PlatformInfoIt->getPlatformPrefix() != PlatformPrefix) {
// This SDK has multiple prefixes, give up and return nothing.
return StringRef();
}
}
return PlatformPrefix;
}

View file

@ -1221,7 +1221,7 @@ AppleMachO::~AppleMachO() {}
MachO::~MachO() {}
void Darwin::VerifyTripleForSDK(const llvm::opt::ArgList &Args,
const llvm::Triple Triple) const {
const llvm::Triple &Triple) const {
if (SDKInfo) {
if (!SDKInfo->supportsTriple(Triple))
getDriver().Diag(diag::warn_incompatible_sysroot)
@ -2042,15 +2042,14 @@ struct DarwinPlatform {
}
static DarwinPlatform createFromSDKInfo(StringRef SDKRoot,
const DarwinSDKInfo &SDKInfo) {
const DarwinSDKInfo::SDKPlatformInfo PlatformInfo =
SDKInfo.getCanonicalPlatformInfo();
const llvm::Triple::OSType OS = PlatformInfo.getOS();
const llvm::Triple &PlatformTriple = SDKInfo.getCanonicalPlatformTriple();
const llvm::Triple::OSType OS = PlatformTriple.getOS();
VersionTuple Version = SDKInfo.getVersion();
if (OS == llvm::Triple::MacOSX)
Version = getVersionFromString(
getSystemOrSDKMacOSVersion(Version.getAsString()));
DarwinPlatform Result(InferredFromSDK, getPlatformFromOS(OS), Version);
Result.Environment = getEnvKindFromEnvType(PlatformInfo.getEnvironment());
Result.Environment = getEnvKindFromEnvType(PlatformTriple.getEnvironment());
Result.InferSimulatorFromArch = false;
Result.InferredSource = SDKRoot;
return Result;
@ -2083,15 +2082,10 @@ struct DarwinPlatform {
llvm::Triple::OSType OS = getOSFromPlatform(Platform);
llvm::Triple::EnvironmentType EnvironmentType =
getEnvTypeFromEnvKind(Environment);
StringRef PlatformPrefix =
(Platform == DarwinPlatformKind::DriverKit) ? "/System/DriverKit" : "";
return DarwinSDKInfo("", OS, EnvironmentType, getOSVersion(),
return DarwinSDKInfo(OS, EnvironmentType, getOSVersion(),
getDisplayName(Platform, Environment, getOSVersion()),
/*MaximumDeploymentTarget=*/
VersionTuple(getOSVersion().getMajor(), 0, 99),
{DarwinSDKInfo::SDKPlatformInfo(
llvm::Triple::Apple, OS, EnvironmentType,
llvm::Triple::MachO, PlatformPrefix)});
VersionTuple(getOSVersion().getMajor(), 0, 99));
}
private:

View file

@ -389,7 +389,7 @@ private:
void AddDeploymentTarget(llvm::opt::DerivedArgList &Args) const;
void VerifyTripleForSDK(const llvm::opt::ArgList &Args,
const llvm::Triple Triple) const;
const llvm::Triple &Triple) const;
protected:
/// Lazily initialize the target platform from the triple when

View file

@ -30,6 +30,10 @@ int main() { return 0; }
// RUN: | FileCheck %s -DSDKROOT=%S/Inputs/DriverKit21.0.1.sdk -DRESOURCE_DIR=%clang-resource-dir --check-prefix=INC
// RUN: %clang %s -target x86_64-apple-driverkit23.0 -isysroot %S/Inputs/DriverKit23.0.sdk -x c++ -### 2>&1 \
// RUN: | FileCheck %s -DSDKROOT=%S/Inputs/DriverKit23.0.sdk -DRESOURCE_DIR=%clang-resource-dir --check-prefix=INC
// RUN: %clang %s -target ppc-apple-driverkit23.0 -isysroot %S/Inputs/DriverKit23.0.sdk -x c++ -### 2>&1 \
// RUN: | FileCheck %s -DSDKROOT=%S/Inputs/DriverKit23.0.sdk -DRESOURCE_DIR=%clang-resource-dir --check-prefix=INC
// RUN: %clang %s -target arm64-apple-driverkit23.0-macabi -isysroot %S/Inputs/DriverKit23.0.sdk -x c++ -### 2>&1 \
// RUN: | FileCheck %s -DSDKROOT=%S/Inputs/DriverKit23.0.sdk -DRESOURCE_DIR=%clang-resource-dir --check-prefix=INC
//
// INC: "-isysroot" "[[SDKROOT]]"
// INC: "-internal-isystem" "[[SDKROOT]]/System/DriverKit/usr/local/include"

View file

@ -1,18 +1,22 @@
// REQUIRES: x86-registered-target
// REQUIRES: arm-registered-target
// REQUIRES: aarch64-registered-target
// RUN: %clang -target x86_64-apple-darwin -Wincompatible-sysroot -isysroot SDKs/MacOSX10.9.sdk -mios-version-min=9.0 -S -o - %s 2>&1 | FileCheck -check-prefix CHECK-OSX-IOS %s
// RUN: %clang -target arm64-apple-darwin -Wincompatible-sysroot -isysroot SDKs/iPhoneOS9.2.sdk -mwatchos-version-min=2.0 -S -o - %s 2>&1 | FileCheck -check-prefix CHECK-IOS-WATCHOS %s
// RUN: %clang -target arm64-apple-darwin -Wincompatible-sysroot -isysroot SDKs/iPhoneOS9.2.sdk -mtvos-version-min=9.0 -S -o - %s 2>&1 | FileCheck -check-prefix CHECK-IOS-TVOS %s
// RUN: %clang -target x86_64-apple-driverkit19.0 -Wincompatible-sysroot -isysroot SDKs/MacOSX10.9.sdk -S -o - %s 2>&1 | FileCheck -check-prefix CHECK-OSX-DRIVERKIT %s
// RUN: %clang -target x86_64-apple-driverkit19.0 -Wincompatible-sysroot -isysroot SDKs/iPhoneOS9.2.sdk -S -o - %s 2>&1 | FileCheck -check-prefix CHECK-IOS-DRIVERKIT %s
// RUN: %clang -target x86_64-apple-darwin -Wincompatible-sysroot -isysroot SDKs/iPhoneSimulator9.2.sdk -mios-version-min=9.0 -S -o - %s 2>&1 | FileCheck -check-prefix CHECK-IOS-IOSSIM %s
// RUN: %clang -target x86_64-apple-darwin -Wno-incompatible-sysroot -isysroot SDKs/MacOSX10.9.sdk -mios-version-min=9.0 -S -o - %s 2>&1 | FileCheck -check-prefix CHECK-OSX-IOS-DISABLED %s
// RUN: %clang -target x86_64-apple-darwin -Wincompatible-sysroot -isysroot SDKs/MacOSX10.9.sdk -mios-version-min=9.0 -### -o - %s 2>&1 | FileCheck -check-prefix CHECK-OSX-IOS %s
// RUN: %clang -target arm64-apple-darwin -Wincompatible-sysroot -isysroot SDKs/iPhoneOS9.2.sdk -mwatchos-version-min=2.0 -### -o - %s 2>&1 | FileCheck -check-prefix CHECK-IOS-WATCHOS %s
// RUN: %clang -target arm64-apple-darwin -Wincompatible-sysroot -isysroot SDKs/iPhoneOS9.2.sdk -mtvos-version-min=9.0 -### -o - %s 2>&1 | FileCheck -check-prefix CHECK-IOS-TVOS %s
// RUN: %clang -target x86_64-apple-driverkit19.0 -Wincompatible-sysroot -isysroot SDKs/MacOSX10.9.sdk -### -o - %s 2>&1 | FileCheck -check-prefix CHECK-OSX-DRIVERKIT %s
// RUN: %clang -target x86_64-apple-driverkit19.0 -Wincompatible-sysroot -isysroot SDKs/iPhoneOS9.2.sdk -### -o - %s 2>&1 | FileCheck -check-prefix CHECK-IOS-DRIVERKIT %s
// RUN: %clang -target x86_64-apple-darwin -Wincompatible-sysroot -isysroot SDKs/iPhoneSimulator9.2.sdk -mios-version-min=9.0 -### -o - %s 2>&1 | FileCheck -check-prefix CHECK-IOS-IOSSIM %s
// RUN: %clang -target x86_64-apple-darwin -Wno-incompatible-sysroot -isysroot SDKs/MacOSX10.9.sdk -mios-version-min=9.0 -### -o - %s 2>&1 | FileCheck -check-prefix CHECK-OSX-IOS-DISABLED %s
// RUN: %clang -target arm64-apple-visionos1.0-simulator -Wincompatible-sysroot -isysroot %S/Inputs/XRSimulator1.0.sdk -S -o - %s 2>&1 | FileCheck -check-prefix CHECK-VISIONOSSIM %s
// RUN: %clang -target arm64-apple-xros1.0 -Wincompatible-sysroot -isysroot %S/Inputs/XRSimulator1.0.sdk -S -o - %s 2>&1 | FileCheck -check-prefix CHECK-VISIONOSSIM-VISIONOS %s
// RUN: %clang -target arm64-apple-ios17.1 -Wincompatible-sysroot -isysroot %S/Inputs/XRSimulator1.0.sdk -S -o - %s 2>&1 | FileCheck -check-prefix CHECK-VISIONOSSIM-IOS %s
// RUN: %clang -target arm64-apple-visionos1.0-simulator -Wincompatible-sysroot -isysroot %S/Inputs/XRSimulator1.0.sdk/usr/include/libxml -S -o - %s 2>&1 | FileCheck -check-prefix CHECK-VISIONOSSIM %s
// RUN: %clang -target armv7k-apple-watchos6.0 -Wincompatible-sysroot -isysroot %S/Inputs/WatchOS6.0.sdk -### -o - %s 2>&1 | FileCheck -check-prefix CHECK-WATCHOS %s
// RUN: %clang -target armv7-apple-watchos6.0 -Wincompatible-sysroot -isysroot %S/Inputs/WatchOS6.0.sdk -### -o - %s 2>&1 | FileCheck -check-prefix CHECK-WATCHOS-ARMV7 %s
// RUN: %clang -target armv6m-apple-firmware -Wincompatible-sysroot -isysroot %S/Inputs/WatchOS6.0.sdk -### -o - %s 2>&1 | FileCheck -check-prefix CHECK-WATCHOS-ARMV6M-FIRMWARE %s
// RUN: %clang -target arm64-apple-visionos1.0-simulator -Wincompatible-sysroot -isysroot %S/Inputs/XRSimulator1.0.sdk -### -o - %s 2>&1 | FileCheck -check-prefix CHECK-VISIONOSSIM %s
// RUN: %clang -target arm64-apple-xros1.0 -Wincompatible-sysroot -isysroot %S/Inputs/XRSimulator1.0.sdk -### -o - %s 2>&1 | FileCheck -check-prefix CHECK-VISIONOSSIM-VISIONOS %s
// RUN: %clang -target arm64-apple-ios17.1 -Wincompatible-sysroot -isysroot %S/Inputs/XRSimulator1.0.sdk -### -o - %s 2>&1 | FileCheck -check-prefix CHECK-VISIONOSSIM-IOS %s
// RUN: %clang -target arm64-apple-visionos1.0-simulator -Wincompatible-sysroot -isysroot %S/Inputs/XRSimulator1.0.sdk/usr/include/libxml -### -o - %s 2>&1 | FileCheck -check-prefix CHECK-VISIONOSSIM %s
int main() { return 0; }
// CHECK-OSX-IOS: warning: using sysroot for 'MacOSX10.9' but targeting 'x86_64-apple-ios9.0.0-simulator'
@ -23,6 +27,9 @@ int main() { return 0; }
// CHECK-IOS-IOSSIM-NOT: warning: using sysroot for '{{.*}}' but targeting '{{.*}}'
// CHECK-OSX-IOS-DISABLED-NOT: warning: using sysroot for '{{.*}}' but targeting '{{.*}}'
// CHECK-WATCHOS-NOT: warning: using sysroot for '{{.*}}' but targeting '{{.*}}'
// CHECK-WATCHOS-ARMV7: warning: using sysroot for 'watchOS 6.0' but targeting 'thumbv7-apple-watchos6.0.0'
// CHECK-WATCHOS-ARMV6M-FIRMWARE: warning: using sysroot for 'watchOS 6.0' but targeting 'thumbv6m-apple-firmware1.0.0'
// CHECK-VISIONOSSIM-NOT: warning: using sysroot for '{{.*}}' but targeting '{{.*}}'
// CHECK-VISIONOSSIM-VISIONOS: warning: using sysroot for 'Simulator - visionOS 1.0' but targeting 'arm64-apple-xros1.0.0'
// CHECK-VISIONOSSIM-IOS: warning: using sysroot for 'Simulator - visionOS 1.0' but targeting 'arm64-apple-ios17.1.0'

View file

@ -107,10 +107,9 @@ TEST(DarwinSDKInfo, PlatformPrefix) {
ASSERT_TRUE(SDKInfo);
EXPECT_EQ(SDKInfo->getPlatformPrefix(Triple("arm64-apple-macos26.0")),
"/System/macOSSupport");
// The triple's architecture doesn't matter.
EXPECT_EQ(SDKInfo->getPlatformPrefix(Triple("ppc-apple-macos26.0")),
"/System/macOSSupport");
// OSes that aren't specified in the SDK never get a system prefix.
// When there are multiple system prefixes, non-matching triples don't get a
// system prefix.
EXPECT_EQ(SDKInfo->getPlatformPrefix(Triple("ppc-apple-macos26.0")), "");
EXPECT_EQ(SDKInfo->getPlatformPrefix(Triple("arm64-apple-ios26.0")), "");
// /System/iOSSupport is not a system prefix, and is ignored.
EXPECT_EQ(SDKInfo->getPlatformPrefix(Triple("arm64-apple-ios26.0-macabi")),