aaron.ballman added a comment.

In https://reviews.llvm.org/D53025#1282423, @ymandel wrote:

> Aaron, Jonas,
>
> Thank you both so much for being fantastic reviewers!  This was my first 
> experience contributing to clang (upstream) and it has been extremely 
> positive.


Congratulations on contributing your first patch!

> In https://reviews.llvm.org/D53025#1282194, @aaron.ballman wrote:
> 
>> This LGTM, thank you for working on this check!
> 
> 
> Great! I don't have SVN commit access.  Can one of you commit this?

I'm happy to do so, but the patch currently does not apply cleanly to the ToT. 
Can you rebase your patch on trunk and update the review with it?


Repository:
  rCTE Clang Tools Extra

https://reviews.llvm.org/D53025



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

Reply via email to