Re: FDO and source changes

[Jeff Law <law@redhat.com>]

On 07/23/14 15:52, Xinliang David Li wrote:
> Index: ChangeLog
> ===================================================================
> --- ChangeLog	(revision 212682)
> +++ ChangeLog	(working copy)
> @@ -1,3 +1,10 @@
> +2014-07-16  Xinliang David Li<davidxl@google.com>
> +
> +	* params.def: New parameter.
> +	* coverage.c (get_coverage_counts): Check new flag.
> +	(coverage_compute_profile_id): Check new flag.
> +	(coverage_begin_function): Check new flag.
> +
>   2014-07-16  Dodji Seketeli<dodji@redhat.com>
>
>   	Support location tracking for built-in macro tokens
> Index: testsuite/ChangeLog
> ===================================================================
> --- testsuite/ChangeLog	(revision 212682)
> +++ testsuite/ChangeLog	(working copy)
> @@ -1,3 +1,11 @@
> +2014-07-16  Xinliang David Li<davidxl@google.com>
> +
> +	* g++.dg/tree-prof/tree-prof.exp: Define macros.
> +	* g++.dg/tree-prof/reorder_class1.h: New file.
> +	* g++.dg/tree-prof/reorder_class2.h: New file.
> +	* g++.dg/tree-prof/reorder.C: New test.
> +	* g++.dg/tree-prof/morefunc.C: New test.
> +
Basically OK.  You need to document the new option in doc/invoke.texi. 
Consider it pre-approved with that addition (please post the final 
version for archival purposes).

Jeff