Path to add -O2 to TORTURE_OPTIONS list.

Horst von Brand vonbrand@inf.utfsm.cl
Thu Feb 25 06:54:00 GMT 1999


Jeffrey A Law <law@hurl.cygnus.com> said:
>   In message < 199902250257.XAA14738@sleipnir.valparaiso.cl >you write:
>   > Nick Clifton <nickc@cygnus.com> said:
>   > >   I have just appleid the patch below to the GCC testsuite, so that
>   > >   the -O3 command line switch will now also be tested.  No new tests
>   > >   or test runs have been introduced by this change.
>   > 
>   > To me it looks like you replaced -O2 by -O3... -O2 _is_ important on
>   > itself, as it is the recommended optimization level for many packages,
>   > including the linux kernel. -O3 isn't used as often because of the
>   > resulting bloat due to indiscriminate inlining.

> If you look more closely, he changed cases which did -O2 -finline-functions 
> ...  to be -O3 -finline-functions.

There is no separate test for just -O2 -fomit-frame-pointer anymore, the
only test left is -O2

> It doesn't actually change the testsuite in any way at the moment, but if
> we ever made -O3 do something other than just turn on funtion inlining,
> then this change will become much more important.

Sorry, I don't buy that argument. If it is the same as before (for now,
until some future unspecified changes to -O3), leave it alone.  Add test
cases for -O3 when they become useful/different.
-- 
Dr. Horst H. von Brand                       mailto:vonbrand@inf.utfsm.cl
Departamento de Informatica                     Fono: +56 32 654431
Universidad Tecnica Federico Santa Maria              +56 32 654239
Casilla 110-V, Valparaiso, Chile                Fax:  +56 32 797513


More information about the Gcc-patches mailing list