This is the mail archive of the
gcc-patches@gcc.gnu.org
mailing list for the GCC project.
Re: [PATCH] Add clobbers for callee copied argument temporaries (PR sanitizer/81715, PR testsuite/83882)
- From: Jakub Jelinek <jakub at redhat dot com>
- To: John David Anglin <dave dot anglin at bell dot net>
- Cc: Richard Biener <rguenther at suse dot de>, gcc-patches at gcc dot gnu dot org
- Date: Thu, 18 Jan 2018 21:34:29 +0100
- Subject: Re: [PATCH] Add clobbers for callee copied argument temporaries (PR sanitizer/81715, PR testsuite/83882)
- Authentication-results: sourceware.org; auth=none
- References: <20180117200749.GX2063@tucnak> <5A609CDA.4060207@bell.net> <20180118175123.GG2063@tucnak> <9f00ac74-3b26-fadc-9af9-b5e6536645a9@bell.net> <0ddd42fa-0adf-a858-37be-476cc0f7ada1@bell.net>
- Reply-to: Jakub Jelinek <jakub at redhat dot com>
On Thu, Jan 18, 2018 at 03:31:38PM -0500, John David Anglin wrote:
> On 2018-01-18 2:44 PM, John David Anglin wrote:
> > > I don't have hpux or hppa-linux fenv.h, so can't check pr52451.c.
> > Will have to investigate. I see they predate your patch.
> This is a test issue. The long double support on hpux is a software
> emulation
> and it's not supported the fenv.h routines.
I think this explanation belongs into the testcase as a comment
to explain why you aren't testing it on hppa-hpux.
Ok with that change.
> 2018-01-18 John David Anglin <danglin@gcc.gnu.org>
>
> * gcc.dg/torture/pr52451.c (main): Skip long double test on
> hppa*-*-hpux*.
>
> Index: gcc.dg/torture/pr52451.c
> ===================================================================
> --- gcc.dg/torture/pr52451.c (revision 256744)
> +++ gcc.dg/torture/pr52451.c (working copy)
> @@ -49,7 +49,9 @@
>
> TEST (float, f);
> TEST (double, );
> +#if !defined(__hppa__) || !defined(__hpux__)
> TEST (long double, l);
> +#endif
>
> return 0;
> }
Jakub