This revision was landed with ongoing or failed builds. This revision was automatically updated to reflect the committed changes. Closed by commit rGbf46b0b55101: [clang][dataflow] Eliminate deprecated `ControlFlowContext::build()` overload. (authored by mboehme).
Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D159262/new/ https://reviews.llvm.org/D159262 Files: clang/include/clang/Analysis/FlowSensitive/ControlFlowContext.h clang/lib/Analysis/FlowSensitive/ControlFlowContext.cpp Index: clang/lib/Analysis/FlowSensitive/ControlFlowContext.cpp =================================================================== --- clang/lib/Analysis/FlowSensitive/ControlFlowContext.cpp +++ clang/lib/Analysis/FlowSensitive/ControlFlowContext.cpp @@ -109,15 +109,5 @@ std::move(BlockReachable)); } -llvm::Expected<ControlFlowContext> -ControlFlowContext::build(const Decl *D, Stmt &S, ASTContext &C) { - if (D == nullptr) - return llvm::createStringError( - std::make_error_code(std::errc::invalid_argument), - "Declaration must not be null"); - - return build(*D, S, C); -} - } // namespace dataflow } // namespace clang Index: clang/include/clang/Analysis/FlowSensitive/ControlFlowContext.h =================================================================== --- clang/include/clang/Analysis/FlowSensitive/ControlFlowContext.h +++ clang/include/clang/Analysis/FlowSensitive/ControlFlowContext.h @@ -40,12 +40,6 @@ static llvm::Expected<ControlFlowContext> build(const Decl &D, Stmt &S, ASTContext &C); - /// Builds a ControlFlowContext from an AST node. `D` is the function in which - /// `S` resides. `D` must not be null and `D->isTemplated()` must be false. - LLVM_DEPRECATED("Use the version that takes a const Decl & instead", "") - static llvm::Expected<ControlFlowContext> build(const Decl *D, Stmt &S, - ASTContext &C); - /// Returns the `Decl` containing the statement used to construct the CFG, if /// available. const Decl &getDecl() const { return ContainingDecl; }
Index: clang/lib/Analysis/FlowSensitive/ControlFlowContext.cpp =================================================================== --- clang/lib/Analysis/FlowSensitive/ControlFlowContext.cpp +++ clang/lib/Analysis/FlowSensitive/ControlFlowContext.cpp @@ -109,15 +109,5 @@ std::move(BlockReachable)); } -llvm::Expected<ControlFlowContext> -ControlFlowContext::build(const Decl *D, Stmt &S, ASTContext &C) { - if (D == nullptr) - return llvm::createStringError( - std::make_error_code(std::errc::invalid_argument), - "Declaration must not be null"); - - return build(*D, S, C); -} - } // namespace dataflow } // namespace clang Index: clang/include/clang/Analysis/FlowSensitive/ControlFlowContext.h =================================================================== --- clang/include/clang/Analysis/FlowSensitive/ControlFlowContext.h +++ clang/include/clang/Analysis/FlowSensitive/ControlFlowContext.h @@ -40,12 +40,6 @@ static llvm::Expected<ControlFlowContext> build(const Decl &D, Stmt &S, ASTContext &C); - /// Builds a ControlFlowContext from an AST node. `D` is the function in which - /// `S` resides. `D` must not be null and `D->isTemplated()` must be false. - LLVM_DEPRECATED("Use the version that takes a const Decl & instead", "") - static llvm::Expected<ControlFlowContext> build(const Decl *D, Stmt &S, - ASTContext &C); - /// Returns the `Decl` containing the statement used to construct the CFG, if /// available. const Decl &getDecl() const { return ContainingDecl; }
_______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits