[PATCH] [MachineSink+PGO] Teach MachineSink to use BlockFrequencyInfo

[Manman Ren]

Hi Bruno,

I am curious why you added a command line option with this updated patch,
is it added just for writing a test case?
Also did you measure any performance change without PGO but with your patch?

Manman

On Wed, Sep 24, 2014 at 12:00 PM, Bruno Cardoso Lopes <
bruno.cardoso at gmail.com> wrote:

> Thanks for the feedback,
>
> Updated the patch to include a test-case.
>
> Manman, they're collected with train data and tested on train data.
> Chad, I've tested it only on darwin x86-64.
>
> http://reviews.llvm.org/D5447
>
> Files:
>   lib/CodeGen/MachineSink.cpp
>   test/CodeGen/X86/sink-blockfreq.ll
>
> _______________________________________________
> llvm-commits mailing list
> llvm-commits at cs.uiuc.edu
> http://lists.cs.uiuc.edu/mailman/listinfo/llvm-commits
>
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20140924/905f4252/attachment.html>