<div dir="ltr">On Sun, Mar 24, 2013 at 7:30 PM, Jiong Wang <span dir="ltr"><<a href="mailto:kasulle@gmail.com" target="_blank" class="cremed">kasulle@gmail.com</a>></span> wrote:<br><div class="gmail_extra"><div class="gmail_quote">
<blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left-width:1px;border-left-color:rgb(204,204,204);border-left-style:solid;padding-left:1ex"><div class="im">on 2013/3/23 1:52, Chris Lattner wrote:<br>

</div><div class="im"><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left-width:1px;border-left-color:rgb(204,204,204);border-left-style:solid;padding-left:1ex">
On Mar 19, 2013, at 8:58 PM, Jiong Wang <<a href="mailto:jiwang@tilera.com" target="_blank" class="cremed">jiwang@tilera.com</a>> wrote:<br>
<br>
<blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left-width:1px;border-left-color:rgb(204,204,204);border-left-style:solid;padding-left:1ex">
Hi Chris,<br>
<br>
could you please comment on committing TILE-Gx backend into community?<br>
</blockquote>
Hi Jiong,<br>
<br>
I don't have any special advice here.  It sounds like the general functionality level is high enough.  Taking it into mainline sounds great, so long as it is reviewed by someone.<br>
</blockquote>
<br></div><div class="im">
  thanks.<br>
<br>
  I will commit after rebasing & re-testing the code on latest llvm mainline. And I will follow <a href="http://llvm.org/docs/HowToAddABuilder.html" target="_blank" class="cremed">http://llvm.org/docs/<u></u>HowToAddABuilder.html</a> to setup a TILE-Gx buildbot.<br>
</div></blockquote><div style><br></div><div style>No, read  what he said again: so long as it is reviewed by someone. That hasn't happened yet, and you need it to happen before you commit.</div><div style><br></div><div style>
As to how you can get the entire backend reviewed, I gave advice on that front in my initial response to your initial email, the very first response you got:</div><div style><br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left-width:1px;border-left-color:rgb(204,204,204);border-left-style:solid;padding-left:1ex">
Unfortunately, this is a *huge* amount of code, and so it is likely to take quite some time to really get reviews on all of it. Please be patient in that regard. </blockquote><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left-width:1px;border-left-color:rgb(204,204,204);border-left-style:solid;padding-left:1ex">
 </blockquote><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left-width:1px;border-left-color:rgb(204,204,204);border-left-style:solid;padding-left:1ex">The best thing you can do to help accelerate the process is to dive in and contribute to the shared parts of the open source project to help the current maintainers, and maybe even free up some of their time to review your patches. Because LLVM is open source, it is really important to the project for folks working on a specific target, backend, or application to also help share the ongoing maintenance and improvement costs of the core shared infrastructure in the compiler.</blockquote>
<div style><br></div><div style>-Chandler</div></div></div></div>