MyDeveloperDay added a comment.

In D75791#2040558 <https://reviews.llvm.org/D75791#2040558>, @MarcusJohnson91 
wrote:

> In D75791#2040532 <https://reviews.llvm.org/D75791#2040532>, @MyDeveloperDay 
> wrote:
>
> > LGTM
>
>
> So what's the next step? I've never committed to LLVM before.


In my mind you have two choices

1. require commit access
2. get someone else to land it

If you think you'd like to continue to be involved and maybe help out from time 
to time (especially with reviews which is where we REALLY lack people) then I 
would recommend 1)

but I'm happy to do 2) for you if not, but you have to be prepared to support 
the patch a little afterwards. (no fire and forget please ;-) )


CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D75791/new/

https://reviews.llvm.org/D75791



_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Reply via email to