[llvm-dev] whatever happened to this patch?

Andrew Kelley via llvm-dev llvm-dev at lists.llvm.org
Wed Apr 15 13:54:32 PDT 2020


Thanks David, that makes sense. So:

1. Find someone who has commit access and knowledge about the touched
areas of code.

2. Convince them to review & land the patch.

On 4/15/20 4:52 PM, David Blaikie wrote:
> 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.
> 
> (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))
> 
> On Wed, Apr 15, 2020 at 1:44 PM Andrew Kelley via llvm-dev
> <llvm-dev at lists.llvm.org <mailto:llvm-dev at lists.llvm.org>> wrote:
> 
>     https://reviews.llvm.org/D62475
> 
>     It looks like it never got merged, what was the missing step to get this
>     landed?
> 
>     Regards,
>     Andrew
> 
>     _______________________________________________
>     LLVM Developers mailing list
>     llvm-dev at lists.llvm.org <mailto:llvm-dev at lists.llvm.org>
>     https://lists.llvm.org/cgi-bin/mailman/listinfo/llvm-dev
> 


-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 833 bytes
Desc: OpenPGP digital signature
URL: <http://lists.llvm.org/pipermail/llvm-dev/attachments/20200415/95ec1caa/attachment.sig>


More information about the llvm-dev mailing list