<div dir="ltr">Generally an author should submit an approved patch. If they do not have commit access they should ask someone who does (usually the approver) to commit it on their behalf.<br><br>(you generally shouldn't approve patches if you aren't in a position to have committed the patch without review if it was your own patch - so at the very least having commit access, but also being sufficiently familiar with the norms of the part of the project that you're confident committing without review (this doesn't fully cover all cases, some areas do more cross-review than others, etc))</div><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">On Wed, Apr 15, 2020 at 1:44 PM Andrew Kelley via llvm-dev <<a href="mailto:llvm-dev@lists.llvm.org" target="_blank">llvm-dev@lists.llvm.org</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex"><a href="https://reviews.llvm.org/D62475" rel="noreferrer" target="_blank">https://reviews.llvm.org/D62475</a><br>
<br>
It looks like it never got merged, what was the missing step to get this<br>
landed?<br>
<br>
Regards,<br>
Andrew<br>
<br>
_______________________________________________<br>
LLVM Developers mailing list<br>
<a href="mailto:llvm-dev@lists.llvm.org" target="_blank">llvm-dev@lists.llvm.org</a><br>
<a href="https://lists.llvm.org/cgi-bin/mailman/listinfo/llvm-dev" rel="noreferrer" target="_blank">https://lists.llvm.org/cgi-bin/mailman/listinfo/llvm-dev</a><br>
</blockquote></div>