alexfh added a comment.

In https://reviews.llvm.org/D41655#1037552, @khuttun wrote:

> In https://reviews.llvm.org/D41655#1037234, @alexfh wrote:
>
> > Do you need help committing the patch?
>
>
> Yes please, I don't have commit access to the repo.


The patch doesn't apply cleanly. Please rebase against current HEAD.

> I think the next step for improving this checker could be to make it work 
> with class template member functions. That would allow to add checking also 
> to those container functions that https://reviews.llvm.org/D17043 handles.

Yep, sounds reasonable. Thank you for working on this!


https://reviews.llvm.org/D41655



_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
  • [PATCH] D41655: [clang... Alexander Kornienko via Phabricator via cfe-commits

Reply via email to