Il 25/09/2012 13:19, Jakub Jelinek ha scritto: > Hi! > > On Thu, Sep 13, 2012 at 06:24:14PM +0200, Paolo Bonzini wrote: >> Il 13/09/2012 17:57, Jakub Jelinek ha scritto: >>>>>>> Can we get this change in? The current state is terribly annoying. >>>>> >>>>> Yes, please go ahead. >>> Here it is, bootstrapped/regtested on x86_64-linux and i686-linux, >>> additionally tested on --disable-bootstrap tree, both by make cc1 inside of >>> gcc subdir (no -O2) and make all-gcc above it (with -O2). >> >> Ok. > > Seems the sed command was using " *" at the end, so it happily changed > e.g. "-Og " into just "g " instead of either keeping "-Og " in, or > removing it altogether. This patches fixes it, now > -Ofast, -Og, -Os, -O, -O[0-9]* are removed when followed by whitespace > and not otherwise. Bootstrapped/regtested on x86_64-linux and i686-linux, > ok for trunk?
Yes, thanks. Would it make sense to leave -Og in? Paolo