I could do it 1 function at a time, but that doesn't seem super useful.  Note that the entire windows code path just delegates to an OS API and can be largely ignored.  So i think it's actually not all that much code.<br><br>I was afraid someone might call for a more thorough review which is why i added dvyukov@<br><br>That said, maybe it's worth looking into how thoroughly it was reviewed before going into LLD?  <br><div class="gmail_quote"><div dir="ltr">On Thu, May 4, 2017 at 2:55 AM Chandler Carruth via Phabricator <<a href="mailto:reviews@reviews.llvm.org">reviews@reviews.llvm.org</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">chandlerc added a comment.<br>
<br>
So, I understand that this is just moving code from LLD to LLVM, but this is pretty complex and subtle code. I think it needs really careful and thorough review. I'm going to try to plan some time for that, but I wonder -- are there any meaningful splits you can make to introduce this more incrementally to LLVM<br>
<br>
<br>
<a href="https://reviews.llvm.org/D32826" rel="noreferrer" target="_blank">https://reviews.llvm.org/D32826</a><br>
<br>
<br>
<br>
</blockquote></div>