Markus, thanks for fixing this during my night.  Appreciate it!

Bill

> On Oct 14, 2016, at 6:08 AM, Markus Trippelsdorf <mar...@trippelsdorf.de> 
> wrote:
> 
> 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  <wschm...@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.
> 
> -- 
> Markus

Reply via email to