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 l...@redhat.com 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

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

2014-09-03 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. Perhaps the

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

2014-09-03 Thread Kaz Kojima
Uros Bizjak ubiz...@gmail.com 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

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 kkoj...@rr.iij4u.or.jp wrote: Uros Bizjak ubiz...@gmail.com wrote: -/* { dg-options -mieee { target sh*-*-* alpha*-*-* } } */ +/* { dg-options -w -mieee { target sh*-*-* alpha*-*-* } } */ /* { dg-skip-if No Inf/NaN support { spu-*-* } * } */

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 oleg.e...@t-online.de 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

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

2014-09-03 Thread Kaz Kojima
Jeff Law l...@redhat.com 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

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

2014-09-02 Thread Kaz Kojima
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] ... It looks that alpha has the similar issue:

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 kkoj...@rr.iij4u.or.jp 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

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

2014-09-02 Thread Kaz Kojima
Oleg Endo oleg.e...@t-online.de 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,