<div dir="ltr">Yup, that's the general idea!<br></div><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">On Wed, Apr 15, 2020 at 1:54 PM Andrew Kelley <<a href="mailto:andrew@ziglang.org" target="_blank">andrew@ziglang.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">Thanks David, that makes sense. So:<br>
<br>
1. Find someone who has commit access and knowledge about the touched<br>
areas of code.<br>
<br>
2. Convince them to review & land the patch.<br>
<br>
On 4/15/20 4:52 PM, David Blaikie wrote:<br>
> Generally an author should submit an approved patch. If they do not have<br>
> commit access they should ask someone who does (usually the approver) to<br>
> commit it on their behalf.<br>
> <br>
> (you generally shouldn't approve patches if you aren't in a position to<br>
> have committed the patch without review if it was your own patch - so at<br>
> the very least having commit access, but also being sufficiently<br>
> familiar with the norms of the part of the project that you're confident<br>
> committing without review (this doesn't fully cover all cases, some<br>
> areas do more cross-review than others, etc))<br>
> <br>
> On Wed, Apr 15, 2020 at 1:44 PM Andrew Kelley via llvm-dev<br>
> <<a href="mailto:llvm-dev@lists.llvm.org" target="_blank">llvm-dev@lists.llvm.org</a> <mailto:<a href="mailto:llvm-dev@lists.llvm.org" target="_blank">llvm-dev@lists.llvm.org</a>>> wrote:<br>
> <br>
>     <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> <mailto:<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>
> <br>
<br>
<br>
</blockquote></div>