<div dir="ltr"><div class="gmail_extra"><br><div class="gmail_quote">On Tue, Jan 13, 2015 at 3:09 PM, Tom Stellard <span dir="ltr"><<a href="mailto:tom@stellard.net" target="_blank">tom@stellard.net</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><span class="">On Tue, Jan 13, 2015 at 03:09:09PM +0000, Renato Golin wrote:<br>
> On 12 January 2015 at 21:07, Alexei Starovoitov<br>
> <<a href="mailto:alexei.starovoitov@gmail.com">alexei.starovoitov@gmail.com</a>> wrote:<br>
> > I'm still not sure who suppose to approve it.<br>
><br>
> Used to be Chris, though it may have changed.<br>
><br>
> If not, I think if Chandler, Eric, Anton, Tom and others are happy<br>
> with, it should be fine, no?<br>
><br>
<br>
</span>There's not a well defined process for adding a new backend, so<br>
usually it ends up being Chris who gives the final approval.  FWIW,<br>
I thinks this is OK to merge.</blockquote></div></div><div class="gmail_extra"><br></div><div class="gmail_extra">I think historically Evan has actually been more involved in shepherding new backends into LLVM.</div><div class="gmail_extra"><br></div><div class="gmail_extra">However, I don't think there is any reason to hold up a new backend if there are multiple active contributors to backends and the common code generator who are happy with this addition and none of the active maintainers speaking up against it or with specific concerns.</div><div class="gmail_extra"><br></div><div class="gmail_extra">(I'd like a chance to glance through the code a bit myself before given explicit support, but will do so soon)</div><div class="gmail_extra"><br></div><div class="gmail_extra">FWIW, I think that once you have a few active backend maintainers who are explicitly supportive, you should just start a *new* thread on llvmdev (so it doesn't get lost with the others) and say "Heads up, BPF backend review is coming to a close over on llvm-commits <link to thread>. It will be landing sometime next week. Just want folks to have a chance to post any comments if they haven't already".</div></div>