[PATCH] D36351: [lld][ELF] Add profile guided section layout

[Deng, Pan via llvm-commits]

Rafael,

Thanks for your help.
Sorry for my late reply, I was just back from holiday.
Yes, I'm using autofdo like pcc, I applied your latest lld patch (from another mail) to Feb 14th's trunk, it works successfully.

Thanks
Pan

-----Original Message-----
From: Rafael Avila de Espindola [mailto:rafael.espindola at gmail.com] 
Sent: Wednesday, February 14, 2018 9:16 AM
To: reviews+D36351+public+19c9952074f60757 at reviews.llvm.org; bigcheesegs at gmail.com; ruiu at google.com; dccitaliano at gmail.com; peter at pcc.me.uk
Cc: arichardson.kde at gmail.com; Deng, Pan <pan.deng at intel.com>; hintonda at gmail.com; george.burgess.iv at gmail.com; joker.eph at gmail.com; hiraditya at msn.com; mgrang at codeaurora.org; zturner at google.com; mgorny at gentoo.org; peter.smith at linaro.org; grimar at accesssoftek.com; simon.f.whittaker at gmail.com; emaste at freebsd.org; llvm-commits at lists.llvm.org; pshung807 at gmail.com; adhemerval.zanella at linaro.org
Subject: Re: [PATCH] D36351: [lld][ELF] Add profile guided section layout

What are you using to build the call graph?

If you are using autofdo like pcc did you only need the lld patch.

Cheers,
Rafael

Pan via Phabricator <reviews at reviews.llvm.org> writes:

> pdeng6 added a comment.
> Herald added a subscriber: arichardson.
>
> @Bigcheese
>
> I'm trying function layout methodologies within ChromeOS to improve chrome performance, this feature is very important since the work relies on it.
> I've checked out the patches for both lld(https://reviews.llvm.org/D36351?id=133550) and llvm(https://reviews.llvm.org/D42161 and https://reviews.llvm.org/D42163), and built it locally,  but I'm afraid the patches for llvm is not as fresh as that for lld.
> Would it be possible for you to show the latest patches at both side,  and the base llvm/lld revisions you are working on? then we can completely align the environment.
>
> thanks
> Pan
>
>
> https://reviews.llvm.org/D36351