VitaNuo added inline comments.
================
Comment at: clang-tools-extra/include-cleaner/lib/WalkAST.cpp:67
bool VisitMemberExpr(MemberExpr *E) {
+ // A member expr implies a usage of the class type
----------------
hokein wrote:
> while reading it again, I realize that there is another case (operator
> overloading) we probably want to handle it as well. It is a more tricky case,
> no need to worry about it in this patch.
>
Thank you. If you'd like me to take care of it, please file an issue with an
example that needs to be taken care of :)
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D139087/new/
https://reviews.llvm.org/D139087
_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
