Rietveld Code Review Tool
Help | Bug tracker | Discussion group | Source code | Sign in
(2357)

Issue 4970041: [pph] Make x7rtti.cc executable (Closed)

Can't Edit
Can't Publish+Mail
Start Review
Created:
13 years, 7 months ago by Diego Novillo
Modified:
13 years, 5 months ago
Reviewers:
CC:
Lawrence Crowl, Gabriel Charette, gcc-patches_gcc.gnu.org
Visibility:
Public.

Patch Set 1 #

Unified diffs Side-by-side diffs Delta from patch set Stats (+12 lines, -7 lines) Patch
M gcc/testsuite/g++.dg/pph/x7rtti.cc View 2 chunks +12 lines, -7 lines 0 comments Download

Messages

Total messages: 1
Diego Novillo
13 years, 7 months ago (2011-08-25 18:51:57 UTC) #1

	* g++.dg/pph/x7rtti.cc: Make it executable.

diff --git a/gcc/testsuite/g++.dg/pph/x7rtti.cc
b/gcc/testsuite/g++.dg/pph/x7rtti.cc
index 297ebe2..0da2f97 100644
--- a/gcc/testsuite/g++.dg/pph/x7rtti.cc
+++ b/gcc/testsuite/g++.dg/pph/x7rtti.cc
@@ -1,3 +1,4 @@
+// { dg-do run }
 // { dg-xfail-if "BOGUS" { "*-*-*" } { "-fpph-map=pph.map" } }
 // { dg-bogus "x7rtti.cc:21:0: warning: .__STDC_IEC_559_COMPLEX__. redefined
.enabled by default." "" { xfail *-*-* } 0 }
 // { dg-bogus "x7rtti.cc:21:0: warning: .__STDC_ISO_10646__. redefined .enabled
by default." "" { xfail *-*-* } 0 }
@@ -13,16 +14,20 @@
 // { dg-bogus "x7rtti.cc:28:1: error: redefinition of .const char
_ZTS15non_polymorphic ..." "" { xfail *-*-* } 0 }
 // { dg-bogus "x7rtti.cc:28:1: error: redefinition of .const char
_ZTS11polymorphic ..." "" { xfail *-*-* } 0 }
 
-
-//FIXME We should make this a run test.
-
 #include "x5rtti1.h"
 #include "x5rtti2.h"
 
 int main()
 {
-    return    poly1() == poly2() && nonp1() == nonp2()
-           && hpol1() == hpol2() && hnpl1() == hnpl2()
-           && poly1() != nonp1() && hpol1() == hnpl1()
-           && poly2() != nonp2() && hpol2() == hnpl2();
+    if (poly1() == poly2()
+	&& nonp1() == nonp2()
+        && hpol1() == hpol2()
+	&& hnpl1() == hnpl2()
+        && poly1() != nonp1()
+	&& hpol1() == hnpl1()
+        && poly2() != nonp2()
+	&& hpol2() == hnpl2())
+      return 0;
+    else
+      return 1;
 }

--
This patch is available for review at http://codereview.appspot.com/4970041
Sign in to reply to this message.

Powered by Google App Engine
RSS Feeds Recent Issues | This issue
This is Rietveld f62528b