zequanwu added inline comments.
================ Comment at: clang/lib/Sema/SemaDeclCXX.cpp:3584-3585 if (!isa<CXXThisExpr>(Base->IgnoreParenImpCasts())) return; ---------------- rtrieu wrote: > Does the warning work if it was changed to be "Visit(Base);" before the > return here instead of the change above? Yes, done. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D85574/new/ https://reviews.llvm.org/D85574 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits