public inbox for gcc-patches@gcc.gnu.org
 help / color / mirror / Atom feed
* Require c99_runtime for builtin-convert-1.c
@ 2015-10-29 15:19 Richard Sandiford
  2015-10-29 15:28 ` Richard Biener
  0 siblings, 1 reply; 2+ messages in thread
From: Richard Sandiford @ 2015-10-29 15:19 UTC (permalink / raw)
  To: gcc-patches

...which was failing on *-elf.  This test is making sure that we narrow
maths calls, e.g. sinh -> sinhf, but we can only do that if we know that
the target has a full c99 libm or if we have other proof that the function
we want to generate is available.

Tested on aarch64-none-elf (where it's skipped) and aarch64-linux-gnu
(where it isn't).  OK to install?

Thanks,
Richard


gcc/testsuite/
	* gcc.dg/torture/builtin-convert-1.c: Require c99_runtime.

diff --git a/gcc/testsuite/gcc.dg/torture/builtin-convert-1.c b/gcc/testsuite/gcc.dg/torture/builtin-convert-1.c
index f13d29e..eecd2b3 100644
--- a/gcc/testsuite/gcc.dg/torture/builtin-convert-1.c
+++ b/gcc/testsuite/gcc.dg/torture/builtin-convert-1.c
@@ -8,6 +8,7 @@
 /* { dg-do link } */
 /* { dg-options "-ffast-math" } */
 /* { dg-add-options c99_runtime } */
+/* { dg-require-effective-target c99_runtime } */
 
 #include "../builtins-config.h"
 

^ permalink raw reply	[flat|nested] 2+ messages in thread

* Re: Require c99_runtime for builtin-convert-1.c
  2015-10-29 15:19 Require c99_runtime for builtin-convert-1.c Richard Sandiford
@ 2015-10-29 15:28 ` Richard Biener
  0 siblings, 0 replies; 2+ messages in thread
From: Richard Biener @ 2015-10-29 15:28 UTC (permalink / raw)
  To: GCC Patches, richard.sandiford

On Thu, Oct 29, 2015 at 4:12 PM, Richard Sandiford
<richard.sandiford@arm.com> wrote:
> ...which was failing on *-elf.  This test is making sure that we narrow
> maths calls, e.g. sinh -> sinhf, but we can only do that if we know that
> the target has a full c99 libm or if we have other proof that the function
> we want to generate is available.
>
> Tested on aarch64-none-elf (where it's skipped) and aarch64-linux-gnu
> (where it isn't).  OK to install?

Ok.

Thanks,
Richard.

> Thanks,
> Richard
>
>
> gcc/testsuite/
>         * gcc.dg/torture/builtin-convert-1.c: Require c99_runtime.
>
> diff --git a/gcc/testsuite/gcc.dg/torture/builtin-convert-1.c b/gcc/testsuite/gcc.dg/torture/builtin-convert-1.c
> index f13d29e..eecd2b3 100644
> --- a/gcc/testsuite/gcc.dg/torture/builtin-convert-1.c
> +++ b/gcc/testsuite/gcc.dg/torture/builtin-convert-1.c
> @@ -8,6 +8,7 @@
>  /* { dg-do link } */
>  /* { dg-options "-ffast-math" } */
>  /* { dg-add-options c99_runtime } */
> +/* { dg-require-effective-target c99_runtime } */
>
>  #include "../builtins-config.h"
>
>

^ permalink raw reply	[flat|nested] 2+ messages in thread

end of thread, other threads:[~2015-10-29 15:25 UTC | newest]

Thread overview: 2+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2015-10-29 15:19 Require c99_runtime for builtin-convert-1.c Richard Sandiford
2015-10-29 15:28 ` Richard Biener

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for read-only IMAP folder(s) and NNTP newsgroup(s).