This is the mail archive of the
gcc-patches@gcc.gnu.org
mailing list for the GCC project.
Re: FDO and source changes
- From: Xinliang David Li <davidxl at google dot com>
- To: Jeff Law <law at redhat dot com>
- Cc: Jan Hubicka <hubicka at ucw dot cz>, GCC Patches <gcc-patches at gcc dot gnu dot org>
- Date: Fri, 25 Jul 2014 15:52:43 -0700
- Subject: Re: FDO and source changes
- Authentication-results: sourceware.org; auth=none
- References: <CAAkRFZ+_sgKGZj07qbkvcw3Emm3Dzmhr0LtLdvnx4nTw2grG=g at mail dot gmail dot com> <53CF2896 dot 1060300 at redhat dot com> <CAAkRFZKa6BY6PFwTaH=EYcUnSf-3yvEj76_8G_tCFUOamTrQLQ at mail dot gmail dot com> <20140723180614 dot GC21250 at kam dot mff dot cuni dot cz> <CAAkRFZJ7Z5DsNf1Zv7Odr=1quJdB0HDf0Bg+tUxBmBBb2DE77w at mail dot gmail dot com> <53D2D265 dot 9090201 at redhat dot com>
Ok. The internal benchmark testing also shows no change in behavior.
David
On Fri, Jul 25, 2014 at 2:55 PM, Jeff Law <law@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<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