public inbox for gcc-patches@gcc.gnu.org
 help / color / mirror / Atom feed
* [patch, testsuite] Fix fragile case nsdmi-union5
@ 2014-04-18  5:30 Joey Ye
  2014-04-21 16:02 ` Mike Stump
  0 siblings, 1 reply; 3+ messages in thread
From: Joey Ye @ 2014-04-18  5:30 UTC (permalink / raw)
  To: gcc-patches

Resulting from discussion here:
http://gcc.gnu.org/ml/gcc/2014-04/msg00125.html

ChangeLog:
	* g++.dg/cpp0x/nsdmi-union5.C: Change to runtime test.

Index: gcc/testsuite/g++.dg/cpp0x/nsdmi-union5.C
===================================================================
--- gcc/testsuite/g++.dg/cpp0x/nsdmi-union5.C	(revision 209462)
+++ gcc/testsuite/g++.dg/cpp0x/nsdmi-union5.C	(working copy)
@@ -1,6 +1,5 @@
 // PR c++/58701
-// { dg-require-effective-target c++11 }
-// { dg-final { scan-assembler "7" } }
+// { dg-do run { target c++11 } }
 
 static union
 {
@@ -9,3 +8,10 @@
     int i = 7;
   };
 };
+
+extern "C" void abort(void);
+int main()
+{
+  if (i != 7) abort();
+  return 0;
+}


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

* Re: [patch, testsuite] Fix fragile case nsdmi-union5
  2014-04-18  5:30 [patch, testsuite] Fix fragile case nsdmi-union5 Joey Ye
@ 2014-04-21 16:02 ` Mike Stump
  2014-04-24  8:27   ` Joey Ye
  0 siblings, 1 reply; 3+ messages in thread
From: Mike Stump @ 2014-04-21 16:02 UTC (permalink / raw)
  To: Joey Ye; +Cc: gcc-patches

On Apr 17, 2014, at 10:28 PM, Joey Ye <joey.ye@arm.com> wrote:
> Resulting from discussion here:
> http://gcc.gnu.org/ml/gcc/2014-04/msg00125.html

Not checked in, and no Ok? asked…  You should do one or the other…  :-)  I’ll assume Ok?

Ok.

> ChangeLog:
> 	* g++.dg/cpp0x/nsdmi-union5.C: Change to runtime test.
> 
> Index: gcc/testsuite/g++.dg/cpp0x/nsdmi-union5.C
> ===================================================================
> --- gcc/testsuite/g++.dg/cpp0x/nsdmi-union5.C	(revision 209462)
> +++ gcc/testsuite/g++.dg/cpp0x/nsdmi-union5.C	(working copy)
> @@ -1,6 +1,5 @@
> // PR c++/58701
> -// { dg-require-effective-target c++11 }
> -// { dg-final { scan-assembler "7" } }
> +// { dg-do run { target c++11 } }
> 
> static union
> {
> @@ -9,3 +8,10 @@
>     int i = 7;
>   };
> };
> +
> +extern "C" void abort(void);
> +int main()
> +{
> +  if (i != 7) abort();
> +  return 0;
> +}

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

* RE: [patch, testsuite] Fix fragile case nsdmi-union5
  2014-04-21 16:02 ` Mike Stump
@ 2014-04-24  8:27   ` Joey Ye
  0 siblings, 0 replies; 3+ messages in thread
From: Joey Ye @ 2014-04-24  8:27 UTC (permalink / raw)
  To: 'Mike Stump'; +Cc: gcc-patches


> -----Original Message-----
> From: Mike Stump [mailto:mikestump@comcast.net]
> Sent: Monday, April 21, 2014 11:39 PM
> To: Joey Ye
> Cc: gcc-patches@gcc.gnu.org
> Subject: Re: [patch, testsuite] Fix fragile case nsdmi-union5
> 
> On Apr 17, 2014, at 10:28 PM, Joey Ye <joey.ye@arm.com> wrote:
> > Resulting from discussion here:
> > http://gcc.gnu.org/ml/gcc/2014-04/msg00125.html
> 
> Not checked in, and no Ok? asked.  You should do one or the other.  :-)
I'll
> assume Ok?
> 
> Ok.
Just committed.


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

end of thread, other threads:[~2014-04-24  8:25 UTC | newest]

Thread overview: 3+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2014-04-18  5:30 [patch, testsuite] Fix fragile case nsdmi-union5 Joey Ye
2014-04-21 16:02 ` Mike Stump
2014-04-24  8:27   ` Joey Ye

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