On Tue, May 13, 2014 at 1:39 AM, Richard Biener <richard.guent...@gmail.com> wrote: > On Fri, May 9, 2014 at 5:54 PM, Teresa Johnson <tejohn...@google.com> wrote: >> I discovered that the support for the documented -fdump-* options >> "optimized", "missed", "note" and "optall" was missing. Added that and >> fixed a minor typo in the documentation. >> >> Bootstrapped and tested on x86-64-unknown-linux-gnu. Ok for trunk? > > I'm not sure they were intented for user-consumption. ISTR they > are just an implementation detail exposed by -fopt-info-X (which is > where they are documented). > > The typo fix is ok, also adding a comment before the dump flags > definition to the above fact. > > David, do I remember correctly?
I remember we talked about content filtering dump flags. Things like -fdump-xxx-ir <-- dump IR only -fdump-xxx-transformation <-- optimization note -fdump-xxx-debug <-- other debug traces Other than that, now I think 'details' and 'all' seem redundant. 'verbose' flag/modifier can achieve the same effect depending on the context. -fdump-xxx-ir-verbose <-- dump IR, and turn on IR modifiers such as vops, lineno, etc -fdump-xxx-transforamtion-verbose <-- dump transformations + missed optimizations + notes -fdump-xxx-debug-verbose <-- turn on detailed trace. thanks, David > > Thanks, > Richard. > >> Thanks, >> Teresa >> >> 2014-05-09 Teresa Johnson <tejohn...@google.com> >> >> * doc/invoke.texi: Fix typo. >> * dumpfile.c: Add support for documented -fdump-* options >> optimized/missed/note/optall. >> >> Index: doc/invoke.texi >> =================================================================== >> --- doc/invoke.texi (revision 210157) >> +++ doc/invoke.texi (working copy) >> @@ -6278,7 +6278,7 @@ passes). >> @item missed >> Enable showing missed optimization information (only available in certain >> passes). >> -@item notes >> +@item note >> Enable other detailed optimization information (only available in >> certain passes). >> @item =@var{filename} >> Index: dumpfile.c >> =================================================================== >> --- dumpfile.c (revision 210157) >> +++ dumpfile.c (working copy) >> @@ -107,6 +107,10 @@ static const struct dump_option_value_info dump_op >> {"nouid", TDF_NOUID}, >> {"enumerate_locals", TDF_ENUMERATE_LOCALS}, >> {"scev", TDF_SCEV}, >> + {"optimized", MSG_OPTIMIZED_LOCATIONS}, >> + {"missed", MSG_MISSED_OPTIMIZATION}, >> + {"note", MSG_NOTE}, >> + {"optall", MSG_ALL}, >> {"all", ~(TDF_RAW | TDF_SLIM | TDF_LINENO | TDF_TREE | TDF_RTL | TDF_IPA >> | TDF_STMTADDR | TDF_GRAPH | TDF_DIAGNOSTIC | TDF_VERBOSE >> | TDF_RHS_ONLY | TDF_NOUID | TDF_ENUMERATE_LOCALS | TDF_SCEV)}, >> >> >> -- >> Teresa Johnson | Software Engineer | tejohn...@google.com | 408-460-2413