On Fri, 12 Jul 2013, Jakub Jelinek wrote:

gcc.c has a huge comment that describes the syntax.

I know, my first reflex after reading your email was to re-read that comment to understand what you had written ;-)

Is the new version ok? Any tests I should do that would be more
useful than running the testsuite? (I tried with several -O options
in various orders and made sure that a -Wl,-On ended up later in the
command line)

I'd say it is important to try say -O17 -O0, -O -O5 etc. and see whether
it works as desired.

Yes, that's what I had checked.

If that works, the patch is ok, with proper ChangeLog entry.

Oups, it was in the first email and I forgot to copy it in the second version.

Thanks,

--
Marc Glisse

Reply via email to