<div>diff --git a/lib/Analysis/BlockFrequencyInfo.cpp b/lib/Analysis/BlockFrequencyInfo.cpp</div><div>index ee89915..9e05065 100644</div><div>--- a/lib/Analysis/BlockFrequencyInfo.cpp</div><div>+++ b/lib/Analysis/BlockFrequencyInfo.cpp</div>

<div>@@ -49,6 +49,10 @@ bool BlockFrequencyInfo::runOnFunction(Function &F) {</div><div>   return false;</div><div> }</div><div> </div><div>+void BlockFrequencyInfo::print(raw_ostream &O, const Module * /*M*/) const {</div>

<div><br></div><div>Just const const Module* will do.</div><div><br></div><div>+  if (BFI) BFI->print(O);</div><div>+}</div><div>+</div><div><br></div><div><div>+; Loop backedges are weighted, and their bodies represented as having a greater</div>

<div>+; frequency.</div></div><div><br></div><div>Please re-english. :)</div><div><br></div><div>Please commit once you've fixed those.</div><div><br></div><div>Nick</div><br><div class="gmail_quote">On 16 October 2011 23:30, 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;">(Sent to the wrong address for Jakub it seems...)<div class="HOEnZb"><div class="h5"><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" target="_blank">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>
</div></div><br>_______________________________________________<br>
llvm-commits mailing list<br>
<a href="mailto:llvm-commits@cs.uiuc.edu">llvm-commits@cs.uiuc.edu</a><br>
<a href="http://lists.cs.uiuc.edu/mailman/listinfo/llvm-commits" target="_blank">http://lists.cs.uiuc.edu/mailman/listinfo/llvm-commits</a><br>
<br></blockquote></div><br>