[llvm-branch-commits] [libcxx] 80878cc - [libc++] Fix XFAIL annotation

[Louis Dionne via llvm-branch-commits]

Author: Louis Dionne
Date: 2021-08-20T10:20:00-04:00
New Revision: 80878ccb3286e7de3ef2f67f24b2026f4544d244

URL: 
https://github.com/llvm/llvm-project/commit/80878ccb3286e7de3ef2f67f24b2026f4544d244
DIFF: 
https://github.com/llvm/llvm-project/commit/80878ccb3286e7de3ef2f67f24b2026f4544d244.diff

LOG: [libc++] Fix XFAIL annotation

The triple can sometimes be arm64-apple-macos, where the previous XFAIL
annotation wouldn't match (and hence the test would fail unexpectedly).

(cherry picked from commit e8118e6c8df52cc026daefc1a1c747d9d8a75ed8)

Added: 


Modified: 
libcxx/test/std/language.support/support.start.term/quick_exit.pass.cpp

Removed: 




diff  --git 
a/libcxx/test/std/language.support/support.start.term/quick_exit.pass.cpp 
b/libcxx/test/std/language.support/support.start.term/quick_exit.pass.cpp
index 83ff44a4f6dc..5a70ea5bd570 100644
--- a/libcxx/test/std/language.support/support.start.term/quick_exit.pass.cpp
+++ b/libcxx/test/std/language.support/support.start.term/quick_exit.pass.cpp
@@ -11,7 +11,7 @@
 // ::quick_exit and ::at_quick_exit are not implemented on macOS.
 // TODO: We should never be using `darwin` as the triple, but LLVM's 
config.guess script
 //   guesses the host triple to be darwin instead of macosx when on macOS.
-// XFAIL: target={{.+}}-apple-macosx{{.+}}
+// XFAIL: target={{.+}}-apple-macos{{.*}}
 // XFAIL: target={{.+}}-apple-darwin{{.+}}
 
 // test quick_exit and at_quick_exit



___
llvm-branch-commits mailing list
llvm-branch-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/llvm-branch-commits

[llvm-branch-commits] [libcxx] 23ba373 - [libc++][PowerPC] Fix a test case failure when compiled with libcxx

[Louis Dionne via llvm-branch-commits]

Author: Albion Fung
Date: 2021-08-20T13:24:44-04:00
New Revision: 23ba3732246a418e52d6d9ca17b45816ade4dc6d

URL: 
https://github.com/llvm/llvm-project/commit/23ba3732246a418e52d6d9ca17b45816ade4dc6d
DIFF: 
https://github.com/llvm/llvm-project/commit/23ba3732246a418e52d6d9ca17b45816ade4dc6d.diff

LOG: [libc++][PowerPC] Fix a test case failure when compiled with libcxx

The test case is not ran unless libcxx is used, and a macro
may be undefined. This patch checks for the definition of the
macro before using it.

Fixes http://llvm.org/PR51430

Differential Revision: https://reviews.llvm.org/D108352

(cherry picked from commit 4bef7a8ff169d6f9a0a253105576ac8a68430702)

Added: 


Modified: 

libcxx/test/std/numerics/numeric.ops/numeric.ops.midpoint/midpoint.float.pass.cpp

Removed: 




diff  --git 
a/libcxx/test/std/numerics/numeric.ops/numeric.ops.midpoint/midpoint.float.pass.cpp
 
b/libcxx/test/std/numerics/numeric.ops/numeric.ops.midpoint/midpoint.float.pass.cpp
index 5cbc40a3578e..4247e2b9e231 100644
--- 
a/libcxx/test/std/numerics/numeric.ops/numeric.ops.midpoint/midpoint.float.pass.cpp
+++ 
b/libcxx/test/std/numerics/numeric.ops/numeric.ops.midpoint/midpoint.float.pass.cpp
@@ -92,7 +92,7 @@ void fp_test()
 assert(d0 < d1);  // sanity checking
 assert(d1 < d2);  // sanity checking
 
-#if defined(__PPC__) && __LONG_DOUBLE_128__ && !__LONG_DOUBLE_IEEE128__
+#if defined(__PPC__) && __LONG_DOUBLE_128__ && 
!(defined(__LONG_DOUBLE_IEEE128__) && __LONG_DOUBLE_IEEE128__)
 // For 128 bit long double implemented as 2 doubles on PowerPC,
 // nextafterl() of libm gives imprecise results which fails the
 // midpoint() tests below. So skip the test for this case.



___
llvm-branch-commits mailing list
llvm-branch-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/llvm-branch-commits