This is the mail archive of the
gcc@gcc.gnu.org
mailing list for the GCC project.
Re: undocumented optimization options
- From: "Richard Guenther" <richard dot guenther at gmail dot com>
- To: "Razya Ladelsky" <RAZYA at il dot ibm dot com>
- Cc: ctice at apple dot com, gcc at gcc dot gnu dot org, gcc-patches at gcc dot gnu dot org, "Gerald Pfeifer" <gerald at pfeifer dot com>, "Janis Johnson" <janis187 at us dot ibm dot com>, "Mark Mitchell" <mark at codesourcery dot com>, stevenb dot gcc at gmail dot com, "Kenneth Zadeck" <zadeck at naturalbridge dot com>
- Date: Mon, 12 Nov 2007 15:33:03 +0100
- Subject: Re: undocumented optimization options
- References: <84fc9c000711120228v1043e0afg548e0a6cd666a7b3@mail.gmail.com> <OF386A4C6B.DEB9E8A8-ONC2257391.004ED087-C2257391.004F4E00@il.ibm.com>
On Nov 12, 2007 3:27 PM, Razya Ladelsky <RAZYA@il.ibm.com> wrote:
> "Richard Guenther" <richard.guenther@gmail.com> wrote on 12/11/2007
> 12:28:05:
>
>
> > On Nov 12, 2007 10:06 AM, Razya Ladelsky <RAZYA@il.ibm.com> wrote:
> > > Kenneth Zadeck <zadeck@naturalbridge.com> wrote on 07/11/2007
> 19:49:29:
> > >
> > > > Razya Ladelsky wrote:
> > > > > Mark Mitchell <mark@codesourcery.com> wrote on 05/11/2007
> 01:51:33:
> > > > >
> > > > >
> > > > >> Gerald Pfeifer wrote:
> > > > >>
> > > > >>> On Thu, 1 Nov 2007, Janis Johnson wrote:
> > > > >>>
> > > > >>>> -fipa-cp steven
> > > > >>>> -fipa-matrix-reorg razya
> > > > >>>> -fipa-pure-const zadeck (enabled with -O)
> > > > >>>> -fipa-reference zadeck (enabled with -O)
> > > > >>>> -fipa-type-escape zadeck
> > > > >>>> -fvar-tracking-uninit ctice
> > > > >>>>
> > > > >>>>
> > > > >
> > > > > I'll add documentation for fipa-cp and fipa-matrix-reorg as soon
> as
> > > > > Zadeck commits his changes to invoke.texi.
> > > > >
> > > > > Thanks,
> > > > > Razya
> > > > >
> > > > >
> > >
> > > Hi,
> > >
> > > This patch adds documentation for fipa-cp and -fipa-matrix-reorg.
> > >
> > > 2007-11-12 Razya Ladelsky <razya@il.ibm.com>
> > >
> > > * doc/invoke.texi (fipa-cp, fipa-matrix-reorg): Add documentation.
> > >
> > > Ok to commit?
> >
> > Please be more verbose. While people might have an idea what
> interprocedural
> > constant propagation is (though they probably don't immediately see that
> it
> > might increase code-size by versioning functions), certainly matrix
> flattening
> > deserves more comments.
> >
>
> I was under the impression that in invoke.texi we just document the
> options.
> The optimizations' full documentation is at the corresponding files.
> (ipa-cp.c and matrix-reorg.c)
> Do you think a more detailed description of these optimizations
> in doc/invoke.texi is necessary?
For options that are not enabled by default at any -O level the documentation
should be a help for the user to decide whether to enable or not enable it.
So, yes, if there are non-standard terms used or if there are not obvious
side-effects (like performing function versioning for ipa-cp) those should be
documented.
For example googling for "interprocedural constant propagation" gives you
an idea what it does by citing papers. Googling for "matrix flattening
transposing" instead gives you only patches ;)
Richard.