Skip to content

Commit f4aa626

Browse files
committed
arm64 macOS was introduced with version Swift
Generalize the check from arm64e to also include arm64. Fixes rdar://83502661.
1 parent b1236e9 commit f4aa626

File tree

1 file changed

+8
-8
lines changed

1 file changed

+8
-8
lines changed

Diff for: lib/Basic/Platform.cpp

+8-8
Original file line numberDiff line numberDiff line change
@@ -375,27 +375,27 @@ swift::getSwiftRuntimeCompatibilityVersionForTarget(
375375
if (Triple.isMacOSX()) {
376376
Triple.getMacOSXVersion(Major, Minor, Micro);
377377

378-
auto floorFor64e = [&Triple](llvm::VersionTuple v) {
379-
if (Triple.getArchName() != "arm64e") return v;
380-
// macOS got first arm64e support in 11.0, i.e. VersionTuple(5, 3)
378+
auto floorFor64 = [&Triple](llvm::VersionTuple v) {
379+
if (!Triple.isAArch64()) return v;
380+
// macOS got first arm64(e) support in 11.0, i.e. VersionTuple(5, 3)
381381
return MAX(v, llvm::VersionTuple(5, 3));
382382
};
383383

384384
if (Major == 10) {
385385
if (Triple.isAArch64() && Minor <= 16)
386-
return floorFor64e(llvm::VersionTuple(5, 3));
386+
return floorFor64(llvm::VersionTuple(5, 3));
387387

388388
if (Minor <= 14) {
389-
return floorFor64e(llvm::VersionTuple(5, 0));
389+
return floorFor64(llvm::VersionTuple(5, 0));
390390
} else if (Minor <= 15) {
391391
if (Micro <= 3) {
392-
return floorFor64e(llvm::VersionTuple(5, 1));
392+
return floorFor64(llvm::VersionTuple(5, 1));
393393
} else {
394-
return floorFor64e(llvm::VersionTuple(5, 2));
394+
return floorFor64(llvm::VersionTuple(5, 2));
395395
}
396396
}
397397
} else if (Major == 11) {
398-
return floorFor64e(llvm::VersionTuple(5, 3));
398+
return floorFor64(llvm::VersionTuple(5, 3));
399399
}
400400
} else if (Triple.isiOS()) { // includes tvOS
401401
Triple.getiOSVersion(Major, Minor, Micro);

0 commit comments

Comments
 (0)