<div dir="ltr"><div class="gmail_quote"><div dir="ltr">On Fri, Oct 12, 2018 at 1:18 PM Yonghong Song via llvm-commits <<a href="mailto:llvm-commits@lists.llvm.org">llvm-commits@lists.llvm.org</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></blockquote><div><br></div><div>I don't know what exactly you meant by "borrowing" code from Linux, but if you simply copy code from Linux to LLVM, and if that code is not available under a compatible license as LLVM's, that will cause trouble. So if you are doing that, please make it very clear that you know what you are doing.</div><div><br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
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>
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>
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>
_______________________________________________<br>
llvm-commits mailing list<br>
<a href="mailto:llvm-commits@lists.llvm.org" target="_blank">llvm-commits@lists.llvm.org</a><br>
<a href="http://lists.llvm.org/cgi-bin/mailman/listinfo/llvm-commits" rel="noreferrer" target="_blank">http://lists.llvm.org/cgi-bin/mailman/listinfo/llvm-commits</a><br>
</blockquote></div></div>