[PATCH, testcase, committed] Exit with zero status from g++.dg/ipa/pr61160-3.C

[PATCH, testcase, committed] Exit with zero status from g++.dg/ipa/pr61160-3.C

[Martin Jambor]

Hi,

in order to avoid spurious testsuite failures, I've checked in the
following obvious patch so that the testcase always returns zero.  I
have verified it still properly tests for non-existence of the bug.

Thanks,

Martin

2014-07-22  Martin Jambor  <mjambor@suse.cz>

	PR ipa/61160
	* g++.dg/ipa/pr61160-3.C (main): Return zero.

Index: gcc/testsuite/g++.dg/ipa/pr61160-3.C
===================================================================
--- gcc/testsuite/g++.dg/ipa/pr61160-3.C	(revision 212914)
+++ gcc/testsuite/g++.dg/ipa/pr61160-3.C	(working copy)
@@ -33,5 +33,6 @@ void *test (MMixin & anExample)
 int main ()
 {
   CExample c;
-  return (test (c) != &c);
+  test (c);
+  return 0;
 }

Re: [PATCH, testcase, committed] Exit with zero status from g++.dg/ipa/pr61160-3.C

[Martin Jambor]

On Tue, Jul 22, 2014 at 06:31:32PM +0200, Martin Jambor wrote:
> Hi,
> 
> in order to avoid spurious testsuite failures, I've checked in the
> following obvious patch so that the testcase always returns zero.  I
> have verified it still properly tests for non-existence of the bug.

...and here goes the same thing for g++.dg/ipa/pr61160-2.C which I
forgot is also a run time test.  I will commit these changes to both
testcase on the 4.9 branch shortly.  Hopefully that will be really it.

Martin

2014-07-24  Martin Jambor  <mjambor@suse.cz>

       PR ipa/61160
       * g++.dg/ipa/pr61160-2.C (main): Always return zero.

Index: g++.dg/ipa/pr61160-2.C
===================================================================
--- g++.dg/ipa/pr61160-2.C      (revision 212986)
+++ g++.dg/ipa/pr61160-2.C      (working copy)
@@ -39,5 +39,6 @@ void *test (MMixin & anExample)
 int main ()
 {
   CExample c;
-  return (test (c) != &c);
+  test (c);
+  return 0;
 }

Re: [PATCH, testcase, committed] Exit with zero status from g++.dg/ipa/pr61160-3.C

[Martin Jambor]

Hi,

apparently I have forgot to backport this to the 4.9 branch which was
now pointed out in a separate PR 61820.  The patch is obvious and
testcase-only, so I have done the backport now, after some basic
testing, so that I can close the bug.

Thanks,

Martin


On Tue, Jul 22, 2014 at 06:31:32PM +0200, Martin Jambor wrote:
> Hi,
> 
> in order to avoid spurious testsuite failures, I've checked in the
> following obvious patch so that the testcase always returns zero.  I
> have verified it still properly tests for non-existence of the bug.
> 
> Thanks,
> 
> Martin
> 
> 2014-07-22  Martin Jambor  <mjambor@suse.cz>
> 
> 	PR ipa/61160
> 	* g++.dg/ipa/pr61160-3.C (main): Return zero.
> 
> Index: gcc/testsuite/g++.dg/ipa/pr61160-3.C
> ===================================================================
> --- gcc/testsuite/g++.dg/ipa/pr61160-3.C	(revision 212914)
> +++ gcc/testsuite/g++.dg/ipa/pr61160-3.C	(working copy)
> @@ -33,5 +33,6 @@ void *test (MMixin & anExample)
>  int main ()
>  {
>    CExample c;
> -  return (test (c) != &c);
> +  test (c);
> +  return 0;
>  }