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?
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
