diff mbox

[PR65767] Fix test case failure on arm-none-eabi

Message ID 000401d07b36$5aec7240$10c556c0$@arm.com
State New
Headers show

Commit Message

Bin Cheng April 20, 2015, 6:50 a.m. UTC
Hi,
As comments at PR65767 and PR65718, we should use namespace other than std
to avoid duplicated definition problem on arm-none-eabi.  This patch fixes
the issue.  It is an obvious change, but I will wait for approval because of
GCC5 branch.

Is it OK?

gcc/testsuite/ChangeLog
2015-04-20  Bin Cheng  <bin.cheng@arm.com>

	PR testsuite/65767
	* g++.dg/lto/pr65276_0.C: Change namespace std to std2.
	* g++.dg/lto/pr65276_1.C: Change namespace std to std2.

Comments

Ramana Radhakrishnan April 20, 2015, 8:17 a.m. UTC | #1
On Mon, Apr 20, 2015 at 7:50 AM, Bin Cheng <bin.cheng@arm.com> wrote:
> Hi,
> As comments at PR65767 and PR65718, we should use namespace other than std
> to avoid duplicated definition problem on arm-none-eabi.  This patch fixes
> the issue.  It is an obvious change, but I will wait for approval because of
> GCC5 branch.

Ok for trunk.

Please wait for Jakub to ack for the GCC 5 branch as the release
appears to be imminent.

Ramana

>
> Is it OK?
>
> gcc/testsuite/ChangeLog
> 2015-04-20  Bin Cheng  <bin.cheng@arm.com>
>
>         PR testsuite/65767
>         * g++.dg/lto/pr65276_0.C: Change namespace std to std2.
>         * g++.dg/lto/pr65276_1.C: Change namespace std to std2.
Jakub Jelinek April 20, 2015, 8:19 a.m. UTC | #2
On Mon, Apr 20, 2015 at 09:17:21AM +0100, Ramana Radhakrishnan wrote:
> On Mon, Apr 20, 2015 at 7:50 AM, Bin Cheng <bin.cheng@arm.com> wrote:
> > Hi,
> > As comments at PR65767 and PR65718, we should use namespace other than std
> > to avoid duplicated definition problem on arm-none-eabi.  This patch fixes
> > the issue.  It is an obvious change, but I will wait for approval because of
> > GCC5 branch.
> 
> Ok for trunk.
> 
> Please wait for Jakub to ack for the GCC 5 branch as the release
> appears to be imminent.

It is ok for the branch even now too.

> > Is it OK?
> >
> > gcc/testsuite/ChangeLog
> > 2015-04-20  Bin Cheng  <bin.cheng@arm.com>
> >
> >         PR testsuite/65767
> >         * g++.dg/lto/pr65276_0.C: Change namespace std to std2.
> >         * g++.dg/lto/pr65276_1.C: Change namespace std to std2.

	Jakub
diff mbox

Patch

diff --git a/gcc/testsuite/g++.dg/lto/pr65276_0.C b/gcc/testsuite/g++.dg/lto/pr65276_0.C
index c8e9699..9c72e68 100644
--- a/gcc/testsuite/g++.dg/lto/pr65276_0.C
+++ b/gcc/testsuite/g++.dg/lto/pr65276_0.C
@@ -7,7 +7,7 @@ 
 
 extern "C++"
 {
-  namespace std
+  namespace std2
   {
     class exception
     {
@@ -16,7 +16,7 @@  extern "C++"
     };
   }
 }
-namespace std
+namespace std2
 {
   struct __cow_string
   {
@@ -31,9 +31,9 @@  namespace std
     __cow_string _M_msg;
   };
 }
-namespace std
+namespace std2
 {
-  class system_error:public std::runtime_error
+  class system_error:public std2::runtime_error
   {
   };
   enum _Ios_Fmtflags
diff --git a/gcc/testsuite/g++.dg/lto/pr65276_1.C b/gcc/testsuite/g++.dg/lto/pr65276_1.C
index ee49752..dafa4fc 100644
--- a/gcc/testsuite/g++.dg/lto/pr65276_1.C
+++ b/gcc/testsuite/g++.dg/lto/pr65276_1.C
@@ -3,7 +3,7 @@ 
 #pragma interface
 extern "C++"
 {
-  namespace std
+  namespace std2
   {
     class exception
     {
@@ -12,7 +12,7 @@  extern "C++"
     };
   }
 }
-namespace std
+namespace std2
 {
   struct __cow_string
   {