diff mbox

[testsuite,avr] : add -finline-limit=0 to pr41885.c options

Message ID 4DA72526.6010304@gjlay.de
State New
Headers show

Commit Message

Georg-Johann Lay April 14, 2011, 4:47 p.m. UTC
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

Comments

Denis Chertykov April 14, 2011, 6:14 p.m. UTC | #1
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.
Richard Biener April 15, 2011, 9:27 a.m. UTC | #2
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.
>
Georg-Johann Lay April 15, 2011, 11:37 a.m. UTC | #3
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
diff mbox

Patch

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>