[PATCH] D17460: Add prefix based function layout when profile is available.

Xinliang David Li via llvm-commits llvm-commits at lists.llvm.org
Mon Feb 22 19:45:57 PST 2016


Actually Hans's original email has links to both the build and test
failures. The followup patch only fixed the build failure, not the
test failure.

David

On Mon, Feb 22, 2016 at 7:14 PM, Dehao Chen <danielcdh at gmail.com> wrote:
> Weird that I received a few buildbot failure emails, and they are fixed in
> http://reviews.llvm.org/D17522
>
> But I didn't receive this one so far.
>
> Sent http://reviews.llvm.org/D17531 to fix this PTAL.
>
> Thanks,
> Dehao
>
> On Mon, Feb 22, 2016 at 6:59 PM, Xinliang David Li <davidxl at google.com>
> wrote:
>>
>> The test case misses mtriple option.
>>
>> Dehao, we should not leave buildbot failing for long period of time.
>> The buildbot message should arrive at your inbox very shortly after
>> the commit -- can you check if there is any issue there (e.g. in spam
>> filter)?
>>
>> thanks,
>>
>> David
>>
>> On Mon, Feb 22, 2016 at 5:50 PM, Amaury SECHET
>> <deadalnix+llvmreview at gmail.com> wrote:
>> > deadalnix added a subscriber: deadalnix.
>> > deadalnix added a comment.
>> >
>> > This is broken on OSX.
>> >
>> >
>> > http://reviews.llvm.org/D17460
>> >
>> >
>> >
>
>


More information about the llvm-commits mailing list