<div dir="ltr"><br><div class="gmail_extra"><br><div class="gmail_quote">On Wed, Jun 10, 2015 at 2:47 AM, David <span dir="ltr"><<a href="mailto:davidxl@google.com" target="_blank" class="cremed">davidxl@google.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">================<br>
Comment at: include/llvm/Analysis/BlockFrequencyInfoImpl.h:1057<br>
@@ -1045,1 +1056,3 @@<br>
+<br>
+    adjustLoopHeaderMass(Loop);<br>
   } else {<br>
----------------<br>
Should this be done iteratively? The mass propagation before this call is based on wrong initial masses for loop headers.<br></blockquote><div><br></div><div>Yeah, iterating a fixed number of times here would give us a better approximation. I was thinking of tuning this after the initial fix.</div><div><br></div><div>Diego.</div></div></div></div>