[llvm-commits] [llvm] r169223 - /llvm/trunk/lib/Transforms/Vectorize/LoopVectorize.cpp

[Chandler Carruth]

On Tue, Dec 4, 2012 at 9:41 AM, Hal Finkel <hfinkel at anl.gov> wrote:

> ----- Original Message -----
> > From: "Nadav Rotem" <nrotem at apple.com>
> > To: llvm-commits at cs.uiuc.edu
> > Sent: Tuesday, December 4, 2012 1:11:52 AM
> > Subject: [llvm-commits] [llvm] r169223 -
>  /llvm/trunk/lib/Transforms/Vectorize/LoopVectorize.cpp
> >
> > Author: nadav
> > Date: Tue Dec  4 01:11:52 2012
> > New Revision: 169223
> >
> > URL: http://llvm.org/viewvc/llvm-project?rev=169223&view=rev
> > Log:
> > Give scalar if-converted blocks half the score because they are not
> > always executed due to CF.
>
> Should we make use of branch weights here? If nothing else,
> LowerExpectIntrinsic transforms uses of __builtin_expect into branch
> weights.
>

Yes, but through BranchProbabilityInfo (or BlockFrequencyInfo, which ever
is more appropriate). if-conversion was a primary intended consumer for
branch probabilities in the new design.
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20121204/f81f99d1/attachment.html>