Re: [RFC] Tweak gcc.c-torture/execute/pr39228.c

2014-09-04 Thread Jeff Law
On 09/03/14 18:22, Kaz Kojima wrote: Jeff Law wrote: * gcc.c-torture/execute/pr39228.c: Use dg-additional-options instead of dg-options and remove sh*-*-* from its target list. Add "inline" keyword to test functions. Wouldn't we be better off moving this into execute/ie

Re: [RFC] Tweak gcc.c-torture/execute/pr39228.c

2014-09-03 Thread Kaz Kojima
Jeff Law wrote: >> * gcc.c-torture/execute/pr39228.c: Use dg-additional-options >> instead of dg-options and remove sh*-*-* from its target list. >> Add "inline" keyword to test functions. > Wouldn't we be better off moving this into execute/ieee? I've tried it and found that all e

Re: [RFC] Tweak gcc.c-torture/execute/pr39228.c

2014-09-03 Thread Jeff Law
On 09/02/14 23:26, Kaz Kojima wrote: Oleg Endo wrote: -mieee should be the default on sh* and thus can be removed from the dg-options line, or is it not? If -mieee is still needed (for alpha) maybe it's better to use dg-additional-options instead? Sure. The attached is a revised one. Reg

Re: [RFC] Tweak gcc.c-torture/execute/pr39228.c

2014-09-03 Thread Richard Biener
On Wed, Sep 3, 2014 at 8:28 AM, Kaz Kojima wrote: > Uros Bizjak wrote: >> -/* { dg-options "-mieee" { target sh*-*-* alpha*-*-* } } */ >> +/* { dg-options "-w -mieee" { target sh*-*-* alpha*-*-* } } */ >> /* { dg-skip-if "No Inf/NaN support" { spu-*-* } "*" "" } */ >> >> Please use /* { dg-add-o

Re: [RFC] Tweak gcc.c-torture/execute/pr39228.c

2014-09-02 Thread Kaz Kojima
Uros Bizjak wrote: > -/* { dg-options "-mieee" { target sh*-*-* alpha*-*-* } } */ > +/* { dg-options "-w -mieee" { target sh*-*-* alpha*-*-* } } */ > /* { dg-skip-if "No Inf/NaN support" { spu-*-* } "*" "" } */ > > Please use /* { dg-add-options ieee } */ directive here. There is > another one p

Re: [RFC] Tweak gcc.c-torture/execute/pr39228.c

2014-09-02 Thread Uros Bizjak
Hello! > It looks that alpha has the similar issue: > https://gcc.gnu.org/ml/gcc-testresults/2014-08/msg02660.html > > alpha and sh redefine dg-options to "-mieee" in the test case > instead of the default dg-options "-w" and get the above warning. > The patch below tweaks the test to fix it. Per

Re: [RFC] Tweak gcc.c-torture/execute/pr39228.c

2014-09-02 Thread Kaz Kojima
Oleg Endo wrote: > -mieee should be the default on sh* and thus can be removed from the > dg-options line, or is it not? If -mieee is still needed (for alpha) maybe > it's better to use dg-additional-options instead? Sure. The attached is a revised one. Regards, kaz -- * gcc.

Re: [RFC] Tweak gcc.c-torture/execute/pr39228.c

2014-09-02 Thread Oleg Endo
Hi, On Sep 3, 2014, at 2:42 AM, Kaz Kojima wrote: > Hi, > > gcc.c-torture/execute/pr39228.c fails with "(test for excess errors)" > on SH for recent revisions. My gcc.log says: > > gcc.c-torture/execute/pr39228.c:20:43: warning: always_inline function might > not be inlinable [-Wattributes]