(Sent to the wrong address for Jakub it seems...)<br><br><div class="gmail_quote">On Sun, Oct 16, 2011 at 11:25 PM, Chandler Carruth <span dir="ltr"><<a href="mailto:chandlerc@google.com">chandlerc@google.com</a>></span> wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex;">Hello,<div><br></div><div>So it turns out that BranchProbabilityInfo and BlockFrequencyInfo aren't rigged up to __builtin_expect at all. I think no one realized because we have essentially no test coverage for these analysis passes. See <a href="http://llvm.org/PR2577" target="_blank">http://llvm.org/PR2577</a></div>

<div><br></div><div>This patch adds test coverage for BlockFrequencyInfo which is the easiest to test, and essentially a good way to strictly cover BranchProbabilityInfo. Subsequent patches to add the remaining features to actually use __builtin_expect will follow with tests based on this.</div>

<div>-Chandler</div>
</blockquote></div><br>