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