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

Hans Wennborg via llvm-commits llvm-commits at lists.llvm.org
Mon Feb 22 14:45:09 PST 2016


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