public inbox for gcc-patches@gcc.gnu.org
 help / color / mirror / Atom feed
* [PATCH, libstdc++] Add missing atomic-builtins argument to experimental/memory_resource/1.cc
@ 2016-05-19 15:03 Thomas Preudhomme
  2016-05-19 21:43 ` Jonathan Wakely
  0 siblings, 1 reply; 2+ messages in thread
From: Thomas Preudhomme @ 2016-05-19 15:03 UTC (permalink / raw)
  To: Jonathan Wakely, gcc-patches

Hi Jonathan,

The dg-require-atomic-builtins in experimental/memory_resource/1.cc does not 
currently work as intended because it is missing its argument. This patch fixes 
that.

ChangeLog entry is as follows:

*** libstdc++-v3/ChangeLog ***

2016-05-18  Thomas Preud'homme  <thomas.preudhomme@arm.com>

        * testsuite/experimental/memory_resource/1.cc: Add required argument
        to dg-require-atomic-builtins.


diff --git a/libstdc++-v3/testsuite/experimental/memory_resource/1.cc 
b/libstdc++-v3/testsuite/experimental/memory_resource/1.cc
index 
22d4e0d966f9c83bb1341137d62c834b441c08f3..08c02e5e31b287cb678c12f499a985899e612748 
100644
--- a/libstdc++-v3/testsuite/experimental/memory_resource/1.cc
+++ b/libstdc++-v3/testsuite/experimental/memory_resource/1.cc
@@ -1,5 +1,5 @@
 // { dg-options "-std=gnu++14" }
-// { dg-require-atomic-builtins }
+// { dg-require-atomic-builtins "" }
 
 // Copyright (C) 2015-2016 Free Software Foundation, Inc.
 //


Is this ok for trunk?

Best regards,

Thomas

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

* Re: [PATCH, libstdc++] Add missing atomic-builtins argument to experimental/memory_resource/1.cc
  2016-05-19 15:03 [PATCH, libstdc++] Add missing atomic-builtins argument to experimental/memory_resource/1.cc Thomas Preudhomme
@ 2016-05-19 21:43 ` Jonathan Wakely
  0 siblings, 0 replies; 2+ messages in thread
From: Jonathan Wakely @ 2016-05-19 21:43 UTC (permalink / raw)
  To: Thomas Preudhomme; +Cc: gcc-patches

On 19 May 2016 at 14:05, Thomas Preudhomme
<thomas.preudhomme@foss.arm.com> wrote:
> Hi Jonathan,
>
> The dg-require-atomic-builtins in experimental/memory_resource/1.cc does not
> currently work as intended because it is missing its argument. This patch fixes
> that.

Oops.

> ChangeLog entry is as follows:
>
> *** libstdc++-v3/ChangeLog ***
>
> 2016-05-18  Thomas Preud'homme  <thomas.preudhomme@arm.com>
>
>         * testsuite/experimental/memory_resource/1.cc: Add required argument
>         to dg-require-atomic-builtins.
>
>
> diff --git a/libstdc++-v3/testsuite/experimental/memory_resource/1.cc
> b/libstdc++-v3/testsuite/experimental/memory_resource/1.cc
> index
> 22d4e0d966f9c83bb1341137d62c834b441c08f3..08c02e5e31b287cb678c12f499a985899e612748
> 100644
> --- a/libstdc++-v3/testsuite/experimental/memory_resource/1.cc
> +++ b/libstdc++-v3/testsuite/experimental/memory_resource/1.cc
> @@ -1,5 +1,5 @@
>  // { dg-options "-std=gnu++14" }
> -// { dg-require-atomic-builtins }
> +// { dg-require-atomic-builtins "" }
>
>  // Copyright (C) 2015-2016 Free Software Foundation, Inc.
>  //
>
>
> Is this ok for trunk?

OK, thanks.

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

end of thread, other threads:[~2016-05-19 21:43 UTC | newest]

Thread overview: 2+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2016-05-19 15:03 [PATCH, libstdc++] Add missing atomic-builtins argument to experimental/memory_resource/1.cc Thomas Preudhomme
2016-05-19 21:43 ` Jonathan Wakely

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