public inbox for gcc-patches@gcc.gnu.org
 help / color / mirror / Atom feed
* [Patch,testsuite,avr]: add -finline-limit=0 to pr41885.c options
@ 2011-04-14 16:47 Georg-Johann Lay
  2011-04-14 18:31 ` Denis Chertykov
  0 siblings, 1 reply; 4+ messages in thread
From: Georg-Johann Lay @ 2011-04-14 16:47 UTC (permalink / raw)
  To: gcc-patches; +Cc: Denis Chertykov, Anatoly Sokolov, Eric Weddington

This patchlet adds -finline-limit=0 to dg-options in

testsuite/gcc.target/avr/torture/pr41885.c

because otherwise optimizers will fold all tests and actually no test
function is called when optimization is on. The test case still passes
all tests.


testsuite/
2011-04-14  Georg-Johann Lay  <avr@gjlay.de>

        * gcc.target/avr/torture/pr41885.c (dg-options): Add
	-finline-limit=0

Index: testsuite/gcc.target/avr/torture/pr41885.c
===================================================================
--- testsuite/gcc.target/avr/torture/pr41885.c  (Revision 172431)
+++ testsuite/gcc.target/avr/torture/pr41885.c  (Arbeitskopie)
@@ -1,4 +1,4 @@
-/* { dg-options "-w -std=c99" } */
+/* { dg-options "-w -std=c99 -finline-limit=0" } */
 /* { dg-do run } */

 #include <limits.h>

^ permalink raw reply	[flat|nested] 4+ messages in thread

* Re: [Patch,testsuite,avr]: add -finline-limit=0 to pr41885.c options
  2011-04-14 16:47 [Patch,testsuite,avr]: add -finline-limit=0 to pr41885.c options Georg-Johann Lay
@ 2011-04-14 18:31 ` Denis Chertykov
  2011-04-15 10:10   ` Richard Guenther
  0 siblings, 1 reply; 4+ messages in thread
From: Denis Chertykov @ 2011-04-14 18:31 UTC (permalink / raw)
  To: Georg-Johann Lay; +Cc: gcc-patches, Anatoly Sokolov, Eric Weddington

2011/4/14 Georg-Johann Lay <avr@gjlay.de>:
> This patchlet adds -finline-limit=0 to dg-options in
>
> testsuite/gcc.target/avr/torture/pr41885.c
>
> because otherwise optimizers will fold all tests and actually no test
> function is called when optimization is on. The test case still passes
> all tests.
>
>
> testsuite/
> 2011-04-14  Georg-Johann Lay  <avr@gjlay.de>
>
>        * gcc.target/avr/torture/pr41885.c (dg-options): Add
>        -finline-limit=0
>
> Index: testsuite/gcc.target/avr/torture/pr41885.c
> ===================================================================
> --- testsuite/gcc.target/avr/torture/pr41885.c  (Revision 172431)
> +++ testsuite/gcc.target/avr/torture/pr41885.c  (Arbeitskopie)
> @@ -1,4 +1,4 @@
> -/* { dg-options "-w -std=c99" } */
> +/* { dg-options "-w -std=c99 -finline-limit=0" } */
>  /* { dg-do run } */
>
>  #include <limits.h>
>

Approved.

Denis.

^ permalink raw reply	[flat|nested] 4+ messages in thread

* Re: [Patch,testsuite,avr]: add -finline-limit=0 to pr41885.c options
  2011-04-14 18:31 ` Denis Chertykov
@ 2011-04-15 10:10   ` Richard Guenther
  2011-04-15 11:52     ` Georg-Johann Lay
  0 siblings, 1 reply; 4+ messages in thread
From: Richard Guenther @ 2011-04-15 10:10 UTC (permalink / raw)
  To: Denis Chertykov
  Cc: Georg-Johann Lay, gcc-patches, Anatoly Sokolov, Eric Weddington

On Thu, Apr 14, 2011 at 8:14 PM, Denis Chertykov <chertykov@gmail.com> wrote:
> 2011/4/14 Georg-Johann Lay <avr@gjlay.de>:
>> This patchlet adds -finline-limit=0 to dg-options in
>>
>> testsuite/gcc.target/avr/torture/pr41885.c
>>
>> because otherwise optimizers will fold all tests and actually no test
>> function is called when optimization is on. The test case still passes
>> all tests.
>>
>>
>> testsuite/
>> 2011-04-14  Georg-Johann Lay  <avr@gjlay.de>
>>
>>        * gcc.target/avr/torture/pr41885.c (dg-options): Add
>>        -finline-limit=0

Please use -fno-inline instead.

Richard.

>> Index: testsuite/gcc.target/avr/torture/pr41885.c
>> ===================================================================
>> --- testsuite/gcc.target/avr/torture/pr41885.c  (Revision 172431)
>> +++ testsuite/gcc.target/avr/torture/pr41885.c  (Arbeitskopie)
>> @@ -1,4 +1,4 @@
>> -/* { dg-options "-w -std=c99" } */
>> +/* { dg-options "-w -std=c99 -finline-limit=0" } */
>>  /* { dg-do run } */
>>
>>  #include <limits.h>
>>
>
> Approved.
>
> Denis.
>

^ permalink raw reply	[flat|nested] 4+ messages in thread

* Re: [Patch,testsuite,avr]: add -finline-limit=0 to pr41885.c options
  2011-04-15 10:10   ` Richard Guenther
@ 2011-04-15 11:52     ` Georg-Johann Lay
  0 siblings, 0 replies; 4+ messages in thread
From: Georg-Johann Lay @ 2011-04-15 11:52 UTC (permalink / raw)
  To: Richard Guenther; +Cc: Denis Chertykov, gcc-patches

Richard Guenther schrieb:
> On Thu, Apr 14, 2011 at 8:14 PM, Denis Chertykov <chertykov@gmail.com> wrote:
>> 2011/4/14 Georg-Johann Lay <avr@gjlay.de>:
>>> This patchlet adds -finline-limit=0 to dg-options in
>>>
>>> testsuite/gcc.target/avr/torture/pr41885.c
>>>
>>> because otherwise optimizers will fold all tests and actually no test
>>> function is called when optimization is on. The test case still passes
>>> all tests.
>>>
>>>
>>> testsuite/
>>> 2011-04-14  Georg-Johann Lay  <avr@gjlay.de>
>>>
>>>        * gcc.target/avr/torture/pr41885.c (dg-options): Add
>>>        -finline-limit=0
> 
> Please use -fno-inline instead.
> 
> Richard.

Ok, changed it:

http://gcc.gnu.org/viewcvs?view=revision&revision=172487

Johann

^ permalink raw reply	[flat|nested] 4+ messages in thread

end of thread, other threads:[~2011-04-15 11:37 UTC | newest]

Thread overview: 4+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2011-04-14 16:47 [Patch,testsuite,avr]: add -finline-limit=0 to pr41885.c options Georg-Johann Lay
2011-04-14 18:31 ` Denis Chertykov
2011-04-15 10:10   ` Richard Guenther
2011-04-15 11:52     ` Georg-Johann Lay

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for read-only IMAP folder(s) and NNTP newsgroup(s).