Ok. The internal benchmark testing also shows no change in behavior. David
On Fri, Jul 25, 2014 at 2:55 PM, Jeff Law <l...@redhat.com> wrote: > 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<davi...@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<do...@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<davi...@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