Bah, Phabricator doesn't show the commits one by one. I will do what I have in the commits where I split up the shadow stuff, then move some code, then split up the rest one by one<span></span>. Will post reviews separately soon.<div><br></div><div>Thank you,</div><div><br></div><div> Filipe<br><br>On Monday, 15 August 2016, Kostya Serebryany <<a href="mailto:kcc@google.com">kcc@google.com</a>> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">kcc added a comment.<br>
<br>
This is a large change and will take quadratic time for review.<br>
Please help me speed it up by splitting the change into several.<br>
Especially, separate simple refactoring (moving code around that does not change functionality) from actual changes in the code (even renaming functions).<br>
<br>
<br>
<a href="https://reviews.llvm.org/D23518" target="_blank">https://reviews.llvm.org/<wbr>D23518</a><br>
<br>
<br>
<br>
</blockquote></div>