diff --git a/clang/lib/ASTMatchers/Dynamic/Marshallers.h b/clang/lib/ASTMatchers/Dynamic/Marshallers.h --- a/clang/lib/ASTMatchers/Dynamic/Marshallers.h +++ b/clang/lib/ASTMatchers/Dynamic/Marshallers.h @@ -925,32 +925,50 @@ const StringRef MatcherName; }; -template class MapAnyOfMatcherDescriptor : public MatcherDescriptor { - std::vector Funcs; + ASTNodeKind CladeNodeKind; + std::vector NodeKinds; public: - MapAnyOfMatcherDescriptor(StringRef MatcherName) - : Funcs{DynCastAllOfMatcherDescriptor( - ast_matchers::internal::VariadicDynCastAllOfMatcher{}, - MatcherName)...} {} + MapAnyOfMatcherDescriptor(ASTNodeKind CladeNodeKind, + std::vector NodeKinds) + : CladeNodeKind(CladeNodeKind), NodeKinds(NodeKinds) {} VariantMatcher create(SourceRange NameRange, ArrayRef Args, Diagnostics *Error) const override { - std::vector InnerArgs; - for (auto const &F : Funcs) { - InnerArgs.push_back(F.create(NameRange, Args, Error)); - if (!Error->errors().empty()) - return {}; + std::vector NodeArgs; + + for (auto NK : NodeKinds) { + std::vector InnerArgs; + + for (const auto &Arg : Args) { + if (!Arg.Value.isMatcher()) + return {}; + const VariantMatcher &VM = Arg.Value.getMatcher(); + if (VM.hasTypedMatcher(NK)) { + auto DM = VM.getTypedMatcher(NK); + InnerArgs.push_back(DM); + } + } + + if (InnerArgs.empty()) { + NodeArgs.push_back( + DynTypedMatcher::trueMatcher(NK).dynCastTo(CladeNodeKind)); + } else { + NodeArgs.push_back( + DynTypedMatcher::constructVariadic( + ast_matchers::internal::DynTypedMatcher::VO_AllOf, NK, + InnerArgs) + .dynCastTo(CladeNodeKind)); + } } - return VariantMatcher::SingleMatcher( - ast_matchers::internal::BindableMatcher( - VariantMatcher::VariadicOperatorMatcher( - ast_matchers::internal::DynTypedMatcher::VO_AnyOf, - std::move(InnerArgs)) - .getTypedMatcher())); + + auto Result = DynTypedMatcher::constructVariadic( + ast_matchers::internal::DynTypedMatcher::VO_AnyOf, CladeNodeKind, + NodeArgs); + Result.setAllowBind(true); + return VariantMatcher::SingleMatcher(Result); } bool isVariadic() const override { return true; } @@ -963,9 +981,11 @@ bool isConvertibleTo(ASTNodeKind Kind, unsigned *Specificity, ASTNodeKind *LeastDerivedKind) const override { - return llvm::all_of(Funcs, [=](const auto &F) { - return F.isConvertibleTo(Kind, Specificity, LeastDerivedKind); - }); + if (Specificity) + *Specificity = 1; + if (LeastDerivedKind) + *LeastDerivedKind = CladeNodeKind; + return true; } }; @@ -1077,8 +1097,9 @@ std::unique_ptr makeMatcherAutoMarshall( ast_matchers::internal::MapAnyOfMatcherImpl, StringRef MatcherName) { - return std::make_unique>( - MatcherName); + return std::make_unique( + ASTNodeKind::getFromNodeKind(), + std::vector{ASTNodeKind::getFromNodeKind()...}); } } // namespace internal