diff --git a/include/clang/ASTMatchers/ASTMatchers.h b/include/clang/ASTMatchers/ASTMatchers.h index ce01f45333b85465b3352d88643ec478b3d47b06..f2c649aca02e94dfd03d0936e9bf3021905f0619 100644 --- a/include/clang/ASTMatchers/ASTMatchers.h +++ b/include/clang/ASTMatchers/ASTMatchers.h @@ -75,18 +75,6 @@ public: return MyBoundNodes.getNodeAs<T>(ID); } - /// \brief Deprecated. Please use \c getNodeAs instead. - /// @{ - template <typename T> - const T *getDeclAs(StringRef ID) const { - return getNodeAs<T>(ID); - } - template <typename T> - const T *getStmtAs(StringRef ID) const { - return getNodeAs<T>(ID); - } - /// @} - /// \brief Type of mapping from binding identifiers to bound nodes. This type /// is an associative container with a key type of \c std::string and a value /// type of \c clang::ast_type_traits::DynTypedNode diff --git a/lib/Tooling/RefactoringCallbacks.cpp b/lib/Tooling/RefactoringCallbacks.cpp index af25fd87b26e760ae909666646761bc9ae7ef0ad..ccc67c44ae993405b7b563f315f3531d926b36fe 100644 --- a/lib/Tooling/RefactoringCallbacks.cpp +++ b/lib/Tooling/RefactoringCallbacks.cpp @@ -39,7 +39,7 @@ ReplaceStmtWithText::ReplaceStmtWithText(StringRef FromId, StringRef ToText) void ReplaceStmtWithText::run( const ast_matchers::MatchFinder::MatchResult &Result) { - if (const Stmt *FromMatch = Result.Nodes.getStmtAs<Stmt>(FromId)) { + if (const Stmt *FromMatch = Result.Nodes.getNodeAs<Stmt>(FromId)) { auto Err = Replace.add(tooling::Replacement( *Result.SourceManager, CharSourceRange::getTokenRange(FromMatch->getSourceRange()), ToText)); @@ -56,8 +56,8 @@ ReplaceStmtWithStmt::ReplaceStmtWithStmt(StringRef FromId, StringRef ToId) void ReplaceStmtWithStmt::run( const ast_matchers::MatchFinder::MatchResult &Result) { - const Stmt *FromMatch = Result.Nodes.getStmtAs<Stmt>(FromId); - const Stmt *ToMatch = Result.Nodes.getStmtAs<Stmt>(ToId); + const Stmt *FromMatch = Result.Nodes.getNodeAs<Stmt>(FromId); + const Stmt *ToMatch = Result.Nodes.getNodeAs<Stmt>(ToId); if (FromMatch && ToMatch) { auto Err = Replace.add( replaceStmtWithStmt(*Result.SourceManager, *FromMatch, *ToMatch)); @@ -75,7 +75,7 @@ ReplaceIfStmtWithItsBody::ReplaceIfStmtWithItsBody(StringRef Id, void ReplaceIfStmtWithItsBody::run( const ast_matchers::MatchFinder::MatchResult &Result) { - if (const IfStmt *Node = Result.Nodes.getStmtAs<IfStmt>(Id)) { + if (const IfStmt *Node = Result.Nodes.getNodeAs<IfStmt>(Id)) { const Stmt *Body = PickTrueBranch ? Node->getThen() : Node->getElse(); if (Body) { auto Err = diff --git a/unittests/AST/DeclPrinterTest.cpp b/unittests/AST/DeclPrinterTest.cpp index dc4f91456071c1ca14508070f7ad4038d5ef07da..e5a09a31f69fc4770f0372c1b732ce1e18760a4e 100644 --- a/unittests/AST/DeclPrinterTest.cpp +++ b/unittests/AST/DeclPrinterTest.cpp @@ -45,7 +45,7 @@ public: PrintMatch() : NumFoundDecls(0) {} void run(const MatchFinder::MatchResult &Result) override { - const Decl *D = Result.Nodes.getDeclAs<Decl>("id"); + const Decl *D = Result.Nodes.getNodeAs<Decl>("id"); if (!D || D->isImplicit()) return; NumFoundDecls++; diff --git a/unittests/AST/StmtPrinterTest.cpp b/unittests/AST/StmtPrinterTest.cpp index bc7fd54e4ad963e7e4069f91c48af97189eda775..12b203236c9966a05463e1a2bb4c1782ab47ac2c 100644 --- a/unittests/AST/StmtPrinterTest.cpp +++ b/unittests/AST/StmtPrinterTest.cpp @@ -45,7 +45,7 @@ public: PrintMatch() : NumFoundStmts(0) {} void run(const MatchFinder::MatchResult &Result) override { - const Stmt *S = Result.Nodes.getStmtAs<Stmt>("id"); + const Stmt *S = Result.Nodes.getNodeAs<Stmt>("id"); if (!S) return; NumFoundStmts++;