<div dir="ltr"><div class="gmail_extra"><br><div class="gmail_quote">On Fri, Apr 11, 2014 at 5:00 PM, Andrew Trick <span dir="ltr"><<a href="mailto:atrick@apple.com" target="_blank">atrick@apple.com</a>></span> wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div id=":fco" class="a3s" style="overflow:hidden">> <<span class="il">block</span>-frequency-20140411.patch><br>

<br>
Thanks. I’m fine if you continue working on this in-tree since you're tracking performance.<br>
<br>
At this point, we mainly want to<br>
- simplify the code by moving helper code into support or eliminating the need for it (especially PositiveFloat)<br>
- add the <span class="il">block</span> bias computation and API<br>
- maybe do a little better on irreducible CFGs</div></blockquote></div><br>LGTM too.</div><div class="gmail_extra"><br></div><div class="gmail_extra">I've looked through the patch, and almost all of the comments are things that really shouldn't be dealt with until the simplifying is done. (stylistic stuff like, using detail namespaces, etc. all of it likely to fall out natrually of the simplifications, no point spending time on it now)</div>
</div>