<div dir="ltr"><br><br><div class="gmail_quote"><div dir="ltr">On Fri, Oct 12, 2018 at 1:18 PM Yonghong Song <<a href="mailto:yhs@fb.com">yhs@fb.com</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><br>
<br>
On 10/12/18 12:48 PM, Eli Friedman via Phabricator wrote:<br>
> efriedma reopened this revision.<br>
> efriedma added a comment.<br>
> This revision is now accepted and ready to land.<br>
> <br>
> Reverted in r344405.<br>
> <br>
> Please review the developer policy again before you commit patches in the future.  Committing patches without review is not allowed except in narrow cases.  And adding new features without tests is not allowed.<br>
<br>
Sorry for a bunch of failures. Some codes are "borrowed" from linux and <br>
I kept the original coding style so we could sync easily but I certainly<br>
should have adopted to llvm coding styles.<br>
<br>
The patch is actual out for a week before the commit and I did not<br>
get any feedback. I will be more active in seeking feedback in<br>
the next revision.<br>
<br></blockquote><div><br></div><div>To elaborate here: you need to wait for approval. There's no "well it's been out for a while" timeout.</div><div> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
I will add tests and fix all the issues fixed by other kind llvm<br>
developers for different platforms, compile flags and then resubmit.<br>
<br></blockquote><div><br></div><div>Please resend for review rather than resubmit.</div><div><br></div><div>Thanks.</div><div><br></div><div>-eric</div><div> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
Sorry again,<br>
<br>
Yonghong<br>
<br>
> <br>
> <br>
> Repository:<br>
>    rL LLVM<br>
> <br>
> <a href="https://urldefense.proofpoint.com/v2/url?u=https-3A__reviews.llvm.org_D52950&d=DwIFaQ&c=5VD0RTtNlTh3ycd41b3MUw&r=DA8e1B5r073vIqRrFz7MRA&m=TX14v0CKfhFaRRizW8e5GMLE7gfjbfXgDOyF46ZFZqs&s=MavpQT_sJ3xX7qXWE3orv2Nb3O65f7jupkRaH5vIPRk&e=" rel="noreferrer" target="_blank">https://urldefense.proofpoint.com/v2/url?u=https-3A__reviews.llvm.org_D52950&d=DwIFaQ&c=5VD0RTtNlTh3ycd41b3MUw&r=DA8e1B5r073vIqRrFz7MRA&m=TX14v0CKfhFaRRizW8e5GMLE7gfjbfXgDOyF46ZFZqs&s=MavpQT_sJ3xX7qXWE3orv2Nb3O65f7jupkRaH5vIPRk&e=</a><br>
> <br>
> <br>
> <br>
</blockquote></div></div>