Author: Sam McCall Date: 2023-03-27T09:29:53+02:00 New Revision: e884e005f36383218641a4512a89a6825059f8cf
URL: https://github.com/llvm/llvm-project/commit/e884e005f36383218641a4512a89a6825059f8cf DIFF: https://github.com/llvm/llvm-project/commit/e884e005f36383218641a4512a89a6825059f8cf.diff LOG: [dataflow] Delete legacy aliases I can't find any trace of use anymore. I'm not sure renaming the header is worth the break, but leave the FIXME. Differential Revision: https://reviews.llvm.org/D146864 Added: Modified: clang/include/clang/Analysis/FlowSensitive/MatchSwitch.h Removed: ################################################################################ diff --git a/clang/include/clang/Analysis/FlowSensitive/MatchSwitch.h b/clang/include/clang/Analysis/FlowSensitive/MatchSwitch.h index 37894ab37dd8d..9a298478c5100 100644 --- a/clang/include/clang/Analysis/FlowSensitive/MatchSwitch.h +++ b/clang/include/clang/Analysis/FlowSensitive/MatchSwitch.h @@ -6,7 +6,7 @@ // //===----------------------------------------------------------------------===// // -// This file defines the `MatchSwitch` abstraction for building a "switch" +// This file defines the `ASTMatchSwitch` abstraction for building a "switch" // statement, where each case of the switch is defined by an AST matcher. The // cases are considered in order, like pattern matching in functional // languages. @@ -17,8 +17,7 @@ // //===----------------------------------------------------------------------===// // -// FIXME: Rename to ASTMatchSwitch.h and update documentation when all usages of -// `MatchSwitch` are updated to `ASTMatchSwitch<Stmt>` +// FIXME: Rename to ASTMatchSwitch.h #ifndef LLVM_CLANG_ANALYSIS_FLOWSENSITIVE_MATCHSWITCH_H_ #define LLVM_CLANG_ANALYSIS_FLOWSENSITIVE_MATCHSWITCH_H_ @@ -69,11 +68,6 @@ template <typename BaseT, typename State, typename Result = void> using ASTMatchSwitch = std::function<Result(const BaseT &, ASTContext &, State &)>; -// FIXME: Remove this alias when all usages of `MatchSwitch` are updated to -// `ASTMatchSwitch<Stmt>`. -template <typename State, typename Result = void> -using MatchSwitch = ASTMatchSwitch<Stmt, State, Result>; - /// Collects cases of a "match switch": a collection of matchers paired with /// callbacks, which together define a switch that can be applied to a node /// whose type derives from `BaseT`. This structure can simplify the definition @@ -171,11 +165,6 @@ class ASTMatchSwitchBuilder { std::vector<MatchSwitchAction<BaseT, State, Result>> Actions; }; -// FIXME: Remove this alias when all usages of `MatchSwitchBuilder` are updated -// to `ASTMatchSwitchBuilder<Stmt>`. -template <typename State, typename Result = void> -using MatchSwitchBuilder = ASTMatchSwitchBuilder<Stmt, State, Result>; - } // namespace dataflow } // namespace clang #endif // LLVM_CLANG_ANALYSIS_FLOWSENSITIVE_MATCHSWITCH_H_ _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
