On Wed, 19 Sep 2012, Jakub Jelinek wrote: > On Wed, Sep 19, 2012 at 03:51:56PM +0200, Richard Guenther wrote: > > Which passes testing now. > > > > Ok? > > Yes. Probably we'll need to retune parallel checking, but we probably need > to do it anyway already now, e.g. i386.exp grew too much.
Committed. Eventually we can shave off some time by removing any of the -O3 options: set C_TORTURE_OPTIONS [list \ { -O0 } \ { -O1 } \ { -O2 } \ { -O3 -fomit-frame-pointer } \ { -O3 -fomit-frame-pointer -funroll-loops } \ { -O3 -fomit-frame-pointer -funroll-all-loops -finline-functions } \ { -O3 -g } \ { -Os } \ { -Og -g } ] -O3 and -Os already enable -finline-functions for example, likewise -fomit-frame-pointer is enabled on most targets. For testing coverage -funroll-all-loops should trump -funroll-loops, and rather than disabling frame-pointers I'd explicitely enable them. Thus keep -O3 -fno-omit-frame-pointer -O3 -funroll-all-loops -O3 -g (not sure why we have -g at -O3 but not at -O[012s] where it should be more common ...) Richard.