hokein accepted this revision. hokein added a comment. This revision is now accepted and ready to land.
Thanks, looks good! please update the description of the patch accordingly (saying we invalidate the vardecl) ================ Comment at: clang/test/SemaCXX/arrow-operator.cpp:79 +void foo() { + // x is dependent. + A<int>& x = blah[7]; // expected-error {{use of undeclared identifier 'blah'}} ---------------- I think the comment `x is dependent` should associate to `x->call()` statement nor the vardecl. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D121824/new/ https://reviews.llvm.org/D121824 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits