This is the mail archive of the
gcc-patches@gcc.gnu.org
mailing list for the GCC project.
Re: [PATCH] Fix up CFLAGS/CXXFLAGS gcc/configure adjustment (PR other/54692)
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