Index: cfe/trunk/include/clang/ASTMatchers/ASTMatchers.h =================================================================== --- cfe/trunk/include/clang/ASTMatchers/ASTMatchers.h +++ cfe/trunk/include/clang/ASTMatchers/ASTMatchers.h @@ -2639,7 +2639,8 @@ /// Overloaded method as shortcut for \c isDerivedFrom(hasName(...)). AST_MATCHER_P_OVERLOAD(CXXRecordDecl, isDerivedFrom, std::string, BaseName, 1) { - assert(!BaseName.empty()); + if (BaseName.empty()) + return false; return isDerivedFrom(hasName(BaseName)).matches(Node, Finder, Builder); } @@ -2655,7 +2656,8 @@ /// \c isSameOrDerivedFrom(hasName(...)). AST_MATCHER_P_OVERLOAD(CXXRecordDecl, isSameOrDerivedFrom, std::string, BaseName, 1) { - assert(!BaseName.empty()); + if (BaseName.empty()) + return false; return isSameOrDerivedFrom(hasName(BaseName)).matches(Node, Finder, Builder); } @@ -2687,7 +2689,8 @@ /// Overloaded method as shortcut for \c isDirectlyDerivedFrom(hasName(...)). AST_MATCHER_P_OVERLOAD(CXXRecordDecl, isDirectlyDerivedFrom, std::string, BaseName, 1) { - assert(!BaseName.empty()); + if (BaseName.empty()) + return false; return isDirectlyDerivedFrom(hasName(BaseName)) .matches(Node, Finder, Builder); } Index: cfe/trunk/unittests/ASTMatchers/ASTMatchersNarrowingTest.cpp =================================================================== --- cfe/trunk/unittests/ASTMatchers/ASTMatchersNarrowingTest.cpp +++ cfe/trunk/unittests/ASTMatchers/ASTMatchersNarrowingTest.cpp @@ -566,6 +566,13 @@ cxxRecordDecl(isDerivedFrom(namedDecl(hasName("X")))))); } +TEST(DeclarationMatcher, IsDerivedFromEmptyName) { + const char *const Code = "class X {}; class Y : public X {};"; + EXPECT_TRUE(notMatches(Code, cxxRecordDecl(isDerivedFrom("")))); + EXPECT_TRUE(notMatches(Code, cxxRecordDecl(isDirectlyDerivedFrom("")))); + EXPECT_TRUE(notMatches(Code, cxxRecordDecl(isSameOrDerivedFrom("")))); +} + TEST(DeclarationMatcher, IsLambda) { const auto IsLambda = cxxMethodDecl(ofClass(cxxRecordDecl(isLambda()))); EXPECT_TRUE(matches("auto x = []{};", IsLambda));