Author: Amara Emerson Date: 2021-01-06T14:57:37-08:00 New Revision: a1265690cf614bde8a7fd1d503c5f13c184dc786
URL: https://github.com/llvm/llvm-project/commit/a1265690cf614bde8a7fd1d503c5f13c184dc786 DIFF: https://github.com/llvm/llvm-project/commit/a1265690cf614bde8a7fd1d503c5f13c184dc786.diff LOG: Fix failing triple test for macOS 11 with non-zero minor versions. Differential Revision: https://reviews.llvm.org/D94197 Added: Modified: llvm/unittests/ADT/TripleTest.cpp llvm/unittests/Support/Host.cpp Removed: ################################################################################ diff --git a/llvm/unittests/ADT/TripleTest.cpp b/llvm/unittests/ADT/TripleTest.cpp index ffce07ba2b12..ff6c2dde4b16 100644 --- a/llvm/unittests/ADT/TripleTest.cpp +++ b/llvm/unittests/ADT/TripleTest.cpp @@ -1264,6 +1264,14 @@ TEST(TripleTest, getOSVersion) { EXPECT_EQ((unsigned)0, Minor); EXPECT_EQ((unsigned)0, Micro); + // For darwin triples on macOS 11, only compare the major version. + T = Triple("x86_64-apple-darwin20.2"); + EXPECT_TRUE(T.isMacOSX()); + T.getMacOSXVersion(Major, Minor, Micro); + EXPECT_EQ((unsigned)11, Major); + EXPECT_EQ((unsigned)0, Minor); + EXPECT_EQ((unsigned)0, Micro); + T = Triple("armv7-apple-ios"); EXPECT_FALSE(T.isMacOSX()); EXPECT_TRUE(T.isiOS()); diff --git a/llvm/unittests/Support/Host.cpp b/llvm/unittests/Support/Host.cpp index 8029bb5830fc..b452048361db 100644 --- a/llvm/unittests/Support/Host.cpp +++ b/llvm/unittests/Support/Host.cpp @@ -348,9 +348,15 @@ TEST_F(HostTest, getMacOSHostVersion) { unsigned HostMajor, HostMinor, HostMicro; ASSERT_EQ(HostTriple.getMacOSXVersion(HostMajor, HostMinor, HostMicro), true); - // Don't compare the 'Micro' version, as it's always '0' for the 'Darwin' - // triples. - ASSERT_EQ(std::tie(SystemMajor, SystemMinor), std::tie(HostMajor, HostMinor)); + if (SystemMajor > 10) { + // Don't compare the 'Minor' and 'Micro' versions, as they're always '0' for + // the 'Darwin' triples on 11.x. + ASSERT_EQ(SystemMajor, HostMajor); + } else { + // Don't compare the 'Micro' version, as it's always '0' for the 'Darwin' + // triples. + ASSERT_EQ(std::tie(SystemMajor, SystemMinor), std::tie(HostMajor, HostMinor)); + } } #endif _______________________________________________ llvm-branch-commits mailing list llvm-branch-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/llvm-branch-commits