public inbox for gcc-patches@gcc.gnu.org
 help / color / mirror / Atom feed
* [PATCH, dfp-testsuite]: Fix PR testsuite/33082
@ 2007-08-16  7:11 Uros Bizjak
  2007-08-16 16:52 ` Janis Johnson
  0 siblings, 1 reply; 2+ messages in thread
From: Uros Bizjak @ 2007-08-16  7:11 UTC (permalink / raw)
  To: gcc-patches

Hello!

-O is not enough to reliably test folding of various dfp values. It is
possible to fix the failure by moving lines of code around as has been
done a couple of weeks ago.

Attached patch fixes this annoyance by simply adding -O2 to folding tests.

Patch was tested on i686-pc-linux-gnu by running DFP testsuite. OK for mainline?

testsuite/ChangeLog:

2007-08-16  Uros Bizjak  <ubizjak@gmail.com>

	PR testsuite/33082
	* gcc.dg/dfp/convert-dfp-fold.c: Use -O2 instead of -O in dg-options.
	* gcc.dg/dfp/convert-bfp-fold.c: Ditto.
	* gcc.dg/dfp/convert-int-fold.c: Ditto.
	* gcc.dg/dfp/operator-arith-fold.c: Ditto.

Uros.

Index: convert-dfp-fold.c
===================================================================
--- convert-dfp-fold.c	(revision 127532)
+++ convert-dfp-fold.c	(working copy)
@@ -1,4 +1,4 @@
-/* { dg-options "-std=gnu99 -O" } */
+/* { dg-options "-std=gnu99 -O2" } */

 /* N1150 5.2 Conversions among decimal floating types and between
    decimal floating types and generic floating types.
Index: convert-int-fold.c
===================================================================
--- convert-int-fold.c	(revision 127532)
+++ convert-int-fold.c	(working copy)
@@ -1,4 +1,4 @@
-/* { dg-options "-std=gnu99 -O" } */
+/* { dg-options "-std=gnu99 -O2" } */

 /* N1150 5.1 Conversion between decimal floating integer.
    C99 6.3.1.4(1a) New.
Index: convert-bfp-fold.c
===================================================================
--- convert-bfp-fold.c	(revision 127532)
+++ convert-bfp-fold.c	(working copy)
@@ -1,4 +1,4 @@
-/* { dg-options "-std=gnu99 -O" } */
+/* { dg-options "-std=gnu99 -O2" } */

 /* N1150 5.2 Conversions among decimal floating types and between
    decimal floating types and generic floating types.
Index: operator-arith-fold.c
===================================================================
--- operator-arith-fold.c	(revision 127532)
+++ operator-arith-fold.c	(working copy)
@@ -1,4 +1,4 @@
-/* { dg-options "-std=gnu99 -O" } */
+/* { dg-options "-std=gnu99 -O2" } */

 /* C99 6.5.5: Multiplicative operators.
    C99 6.5.6: Additive operators.  */

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

* Re: [PATCH, dfp-testsuite]: Fix PR testsuite/33082
  2007-08-16  7:11 [PATCH, dfp-testsuite]: Fix PR testsuite/33082 Uros Bizjak
@ 2007-08-16 16:52 ` Janis Johnson
  0 siblings, 0 replies; 2+ messages in thread
From: Janis Johnson @ 2007-08-16 16:52 UTC (permalink / raw)
  To: Uros Bizjak; +Cc: gcc-patches

On Thu, 2007-08-16 at 09:11 +0200, Uros Bizjak wrote:
> Hello!
> 
> -O is not enough to reliably test folding of various dfp values. It is
> possible to fix the failure by moving lines of code around as has been
> done a couple of weeks ago.
> 
> Attached patch fixes this annoyance by simply adding -O2 to folding tests.
> 
> Patch was tested on i686-pc-linux-gnu by running DFP testsuite. OK for mainline?

OK.

Janis

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

end of thread, other threads:[~2007-08-16 16:52 UTC | newest]

Thread overview: 2+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2007-08-16  7:11 [PATCH, dfp-testsuite]: Fix PR testsuite/33082 Uros Bizjak
2007-08-16 16:52 ` Janis Johnson

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).