This is the mail archive of the
gcc-patches@gcc.gnu.org
mailing list for the GCC project.
Re: [PATCH] Test cases for PR77937
- From: Rainer Orth <ro at CeBiTec dot Uni-Bielefeld dot DE>
- To: Markus Trippelsdorf <markus at trippelsdorf dot de>
- Cc: Bill Schmidt <wschmidt at linux dot vnet dot ibm dot com>, gcc-patches at gcc dot gnu dot org
- Date: Fri, 14 Oct 2016 13:28:22 +0200
- Subject: Re: [PATCH] Test cases for PR77937
- Authentication-results: sourceware.org; auth=none
- References: <1476388387.5711.8.camel@oc8801110288.ibm.com> <yddfuo0ow13.fsf@CeBiTec.Uni-Bielefeld.DE> <20161014110808.GB306@x4>
Hi Markus,
> On 2016.10.13 at 22:04 +0200, Rainer Orth wrote:
>> Hi Bill,
>>
>> > Here are torture test cases for
>> > https://gcc.gnu.org/bugzilla/show_bug.cgi?id=77937. Markus Trippelsdorf
>> > kindly provided the source for the tests and verified the correct
>> > dejagnu options on x86_64-pc-linux-gnu. Committed.
>> >
>> > Thanks,
>> > Bill
>> >
>> >
>> > 2016-10-13 Bill Schmidt <wschmidt@linux.vnet.ibm.com>
>> >
>> > PR tree-optimization/77937
>> > * gcc.dg/torture/pr77937-1.c: New.
>> > * gcc.dg/torture/pr77937-2.c: New.
>> >
>> >
>> > Index: gcc/testsuite/gcc.dg/torture/pr77937-1.c
>> > ===================================================================
>> > --- gcc/testsuite/gcc.dg/torture/pr77937-1.c (revision 0)
>> > +++ gcc/testsuite/gcc.dg/torture/pr77937-1.c (working copy)
>> > @@ -0,0 +1,14 @@
>> > +/* { dg-do compile } */
>> > +/* { dg-do options "-O3 -march=amdfam10" { target { x86_64-*-* } } } */
>>
>> this can't be right: you always need target { i?86-*-* x86_64-*-* } and
>> if really need be restrict it to 64-bit only with lp64. This makes sure
>> the test is run correctly for multilib x86 configurations
>> (e.g. i686-pc-linux-gnu with -m64). Same in the other test.
>
> This was in part because of my sloppy review. Sorry.
> I've committed the fix as obvious.
You should have added a ChangeLog entry for that change. Unfortunately,
neither the original testcases nor the fixed ones can ever have been
properly tested: there's no such thing as dg-do options, leading to
horrendous errors in mail-report.log:
ERROR: gcc.dg/torture/pr77937-1.c -O0 : 2: too many arguments for " dg-do 2 options "-O3 -march=amdfam10" { target { x86_64-*-* } } "
UNRESOLVED: gcc.dg/torture/pr77937-1.c -O0 : 2: too many arguments for " dg-do 2 options "-O3 -march=amdfam10" { target { x86_64-*-* } } "
I missed that myself ;-(
I've now applied Uros' patch merged with yours, with this ChangeLog
entry after proper testing of both multilibs on x86_64-pc-linux-gnu.
Rainer
--
-----------------------------------------------------------------------------
Rainer Orth, Center for Biotechnology, Bielefeld University
2016-10-14 Uros Bizjak <ubizjak@gmail.com>
* gcc.dg/torture/pr77937-1.c: Move -O3 to dg-options.
Pass -march=amdfam10 for both i?86-?-* and x86_64-*-*.
* gcc.dg/torture/pr77937-2.c: Likewise.
diff --git a/gcc/testsuite/gcc.dg/torture/pr77937-1.c b/gcc/testsuite/gcc.dg/torture/pr77937-1.c
--- a/gcc/testsuite/gcc.dg/torture/pr77937-1.c
+++ b/gcc/testsuite/gcc.dg/torture/pr77937-1.c
@@ -1,5 +1,6 @@
/* { dg-do compile } */
-/* { dg-do options "-O3 -march=amdfam10" { target { i?86-*-* x86_64-*-* } } } */
+/* { dg-options "-O3" } */
+/* { dg-additional-options "-march=amdfam10" { target i?86-*-* x86_64-*-* } } */
int *a;
int b, c, d;
diff --git a/gcc/testsuite/gcc.dg/torture/pr77937-2.c b/gcc/testsuite/gcc.dg/torture/pr77937-2.c
--- a/gcc/testsuite/gcc.dg/torture/pr77937-2.c
+++ b/gcc/testsuite/gcc.dg/torture/pr77937-2.c
@@ -1,5 +1,6 @@
/* { dg-do compile } */
-/* { dg-do options "-O3 -march=amdfam10" { target { i?86-*-* x86_64-*-* } } } */
+/* { dg-options "-O3" } */
+/* { dg-additional-options "-march=amdfam10" { target i?86-*-* x86_64-*-* } } */
extern int fn2(int);
extern int fn3(int);