This is the mail archive of the
gcc@gcc.gnu.org
mailing list for the GCC project.
Re: AutoFDO profile toolchain is open-sourced
- From: Dehao Chen <dehao at google dot com>
- To: Jan Hubicka <hubicka at ucw dot cz>
- Cc: Andi Kleen <andi at firstfloor dot org>, Ilya Palachev <i dot palachev at samsung dot com>, Diego Novillo <dnovillo at google dot com>, GCC Development <gcc at gcc dot gnu dot org>, David Li <davidxl at google dot com>, Sebastian Pop <sebpop at gmail dot com>, Vyacheslav Barinov <v dot barinov at samsung dot com>
- Date: Mon, 27 Apr 2015 09:34:59 -0700
- Subject: Re: AutoFDO profile toolchain is open-sourced
- Authentication-results: sourceware.org; auth=none
- References: <553653CA dot 9070009 at samsung dot com> <87wq15mud5 dot fsf at tassilo dot jf dot intel dot com> <CAO2gOZW2Qmp-xj7Cx-7pDm9wg8BO7JEXvi9z_2_-3ms0bRmLeg at mail dot gmail dot com> <20150421172436 dot GD2366 at two dot firstfloor dot org> <CAO2gOZUWVr_W4REyq-PT+O9-0yoWc9Je0eMTs1euSZp6kdR==Q at mail dot gmail dot com> <20150421173630 dot GE2366 at two dot firstfloor dot org> <CAO2gOZUDdDAUFczkjTCFbkyXPuS5NVTrAyA+Vk1Lmpy_A8hrbg at mail dot gmail dot com> <20150422031547 dot GF2366 at two dot firstfloor dot org> <20150422034704 dot GG2366 at two dot firstfloor dot org> <CAO2gOZWvBJmt6LBy9BVH_NScUXydu3ZUqWBWbaY1iY4jiypbXA at mail dot gmail dot com> <20150424053154 dot GA719 at atrey dot karlin dot mff dot cuni dot cz>
On Thu, Apr 23, 2015 at 10:31 PM, Jan Hubicka <hubicka@ucw.cz> wrote:
>
> > > It converts with the attached patches, but there's still some problem
> > > parsing the data:
> > >
> > > % ./create_gcov -binary loop -gcov_version 1 -gcov loop.gcda -gcov_version 0x500e
> > > % gcc50 -O2 -fprofile-use loop.c
> > > loop.c:1:0: warning: '/home/andi/src/autofdo/loop.gcda' is version ',
> > > expected version '500e'
> > > %
> >
> > You need to use -fauto-profile=loop.gcda instead of "-fprofile-use",
> > which is only for instrumentation based FDO.
>
> This is indeed not very intuitive. I wonder why it uses the same suffix suggesting
> that sample based and FDO based files are the same?
AutoFDO profile does not need to have any specific suffix. I'll update
the toolchain to make the default output profile as "fbdata.afdo"
instead of "fbdata.gcda".
>
> Would it be possible to at least have this well documented in invoke.texi and perhaps
> we can fix the warning above to say something like "loop.gcda is autofdo profile, use
> -fauto-profile instead of -fprofile-use"?
Sounds good to me. I will send a patch to update invoke.texi. Could
you help fix the warning for profile-use?
Thanks,
Dehao
>
>
> Honza
> >
> > Dehao
> >
> > >
> > > -Andi
> > >