This is the mail archive of the
gcc-patches@gcc.gnu.org
mailing list for the GCC project.
Re: [PATCH][testsuite] Don't run cproj-fails-with-broken-glibc.c for broken glibc
- From: Rainer Orth <ro at CeBiTec dot Uni-Bielefeld dot DE>
- To: Tom de Vries <Tom_deVries at mentor dot com>
- Cc: GCC Patches <gcc-patches at gcc dot gnu dot org>, Mike Stump <mikestump at comcast dot net>
- Date: Fri, 01 Aug 2014 12:35:15 +0200
- Subject: Re: [PATCH][testsuite] Don't run cproj-fails-with-broken-glibc.c for broken glibc
- Authentication-results: sourceware.org; auth=none
- References: <53DA9714 dot 5070707 at mentor dot com>
Hi Tom,
> The test-case cproj-fails-with-broken-glibc.c does not work with broken
> glibcs, as the header comment mentions:
> ...
> Check the runtime behavior of the C library's cproj() function and
> whether it follows the standard. Versions of GLIBC through 2.11.1
> had an incorrect implementation which will conflict with GCC's
> builtin cproj(). GLIBC 2.12+ should be okay.
> ...
>
> This patch skips the test for known broken glibcs. OK for trunk?
I'm not at all happy with this patch: unless absolutely necessary, we
shouldn't check for version numbers when there's any way to check for
working/broken features instead.
I'm all for keeping testsuite results clean, but this seems like a total
corner case to me. Which distributions do still use affected glibc
versions?
That test, even if we go the glibc version route, needs to be XFAILed
instead of requiring the working version. Apart from that, new
effective-target keywords need documenting in doc/sourcebuild.texi.
Thanks.
Rainer
--
-----------------------------------------------------------------------------
Rainer Orth, Center for Biotechnology, Bielefeld University