<div dir="ltr"><div class="gmail_extra"><br><div class="gmail_quote">On Wed, Aug 13, 2014 at 3:49 PM, Gerolf Hoflehner <span dir="ltr"><<a href="mailto:ghoflehner@apple.com" target="_blank">ghoflehner@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=":cyk" class="a3s" style="overflow:hidden">GTM-24hr. If you don’t get a review within 24hr make mine count :-)<br>

<br>
My disclosure should make people nervous and get you a higher quality review, though:<br>
1) I’m not an expert in the InstCombine code<br>
2) Even better, it is the first time I have ever I looked into that code<br>
3) Yi’s original logic looked OK to me already until Nuno and his magic tool uncovered<br>
all the subtle conditions under which his transformation holds.</div></blockquote></div><br>FWIW, please don't give time limited LGTMs. If you're not comfortable OK-ing the patch to go in, don't OK it. Your review comments and feedback are still helpful and its easy to say "this looks good to me, but wait for someone with more <whatever> to give the final OK".</div>
</div>