[llvm] r261582 - Add prefix based function layout when profile is available.

Dehao Chen via llvm-commits llvm-commits at lists.llvm.org
Mon Feb 22 14:55:26 PST 2016


Fix on the way: http://reviews.llvm.org/D17524

On Mon, Feb 22, 2016 at 2:45 PM, Hans Wennborg <hans at chromium.org> wrote:
> For example:
> http://bb.pgr.jp/builders/i686-mingw32-RA-on-linux/builds/8010/
>
> Other bots (those that link statically I suspect) fail on the test instead:
> http://bb.pgr.jp/builders/ninja-x64-msvc-RA-centos6/builds/20319
> http://lab.llvm.org:8080/green/job/clang-stage1-cmake-RA-incremental_check/19399/
>
> On Mon, Feb 22, 2016 at 2:42 PM, Dehao Chen <dehao at google.com> wrote:
>> Thanks for the heads up. Could you point me to where I can check the
>> buildbot failures?
>>
>> Dehao
>>
>> On Mon, Feb 22, 2016 at 2:38 PM, Hans Wennborg <hans at chromium.org> wrote:
>>> On Mon, Feb 22, 2016 at 2:14 PM, Dehao Chen via llvm-commits
>>> <llvm-commits at lists.llvm.org> wrote:
>>>> Author: dehao
>>>> Date: Mon Feb 22 16:14:14 2016
>>>> New Revision: 261582
>>>>
>>>> URL: http://llvm.org/viewvc/llvm-project?rev=261582&view=rev
>>>> Log:
>>>> Add prefix based function layout when profile is available.
>>>>
>>>> Summary: If a function is hot, put it in text.hot section.
>>>>
>>>> Reviewers: davidxl
>>>>
>>>> Subscribers: eraman, mcrosier, llvm-commits
>>>>
>>>> Differential Revision: http://reviews.llvm.org/D17460
>>>
>>> Many buildbots are broken. I think you'll need to add ProfileData to
>>> the required_libraries in lib/CodeGen/LLVMBuild.txt. (Unless this
>>> would create some kind of layering violation.)


More information about the llvm-commits mailing list