Hi! As mentioned in the PR, I believe we should use long double float suffixes in the test testing long double and Q suffixes in the test that tests __float128, both because PowerPC doesn't allow mixing them and because only the latter test is guarded on float128 support.
Bootstrapped/regtested on x86_64-linux and i686-linux, committed to trunk. 2019-12-18 Jakub Jelinek <ja...@redhat.com> PR middle-end/86416 * testsuite/libgomp.c/pr86416-1.c (main): Use L suffixes rather than q or none. * testsuite/libgomp.c/pr86416-2.c (main): Use Q suffixes rather than L or none. --- libgomp/testsuite/libgomp.c/pr86416-1.c.jj 2019-12-18 21:25:02.856131826 +0100 +++ libgomp/testsuite/libgomp.c/pr86416-1.c 2019-12-18 21:28:06.275349386 +0100 @@ -16,7 +16,7 @@ long double foo (long double x) int main() { - long double v = foo (10.0q) - 20.0q; - if (v > 1.0e-5 || v < -1.0e-5) abort(); + long double v = foo (10.0L) - 20.0L; + if (v > 1.0e-5L || v < -1.0e-5L) abort(); return 0; } --- libgomp/testsuite/libgomp.c/pr86416-2.c.jj 2019-12-18 21:25:02.855131842 +0100 +++ libgomp/testsuite/libgomp.c/pr86416-2.c 2019-12-18 21:28:41.708811864 +0100 @@ -16,7 +16,7 @@ __float128 foo(__float128 y) int main() { - __float128 v = foo (5.0L) - 20.0L; - if (v > 1.0e-5 || v < -1.0e-5) abort(); + __float128 v = foo (5.0Q) - 20.0Q; + if (v > 1.0e-5Q || v < -1.0e-5Q) abort(); return 0; } Jakub