klimek added inline comments. ================ Comment at: clang-rename/USRFinder.h:28-33 @@ -27,6 +27,8 @@ -// Given an AST context and a point, returns a NamedDecl identifying the symbol -// at the point. Returns null if nothing is found at the point. +// Given an AST context and a point (or fully qualified name), returns a +// NamedDecl identifying the symbol at the point. Returns null if nothing is +// found at the point. const NamedDecl *getNamedDeclAt(const ASTContext &Context, - const SourceLocation Point); + const SourceLocation Point, + const std::string &Name); ---------------- These seem to be fundamentally different use cases. Can we add a new top-level function instead? It seems like the new case would have a much simpler implementation.
http://reviews.llvm.org/D21517 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits