[clang] [clang-tools-extra] [llvm] Add ``ignoringParenImpCasts`` in arguments of hasArgument (PR #89553)

Julian Schmidt via llvm-commits llvm-commits at lists.llvm.org
Wed May 1 12:23:42 PDT 2024


================
@@ -82,8 +82,8 @@ void StringConstructorCheck::registerMatchers(MatchFinder *Finder) {
   Finder->addMatcher(
       cxxConstructExpr(
           hasDeclaration(cxxMethodDecl(hasName("basic_string"))),
-          hasArgument(0, hasType(qualType(isInteger()))),
-          hasArgument(1, hasType(qualType(isInteger()))),
+          hasArgument(0, ignoringParenImpCasts(hasType(qualType(isInteger())))),
+          hasArgument(1, ignoringParenImpCasts(hasType(qualType(isInteger())))),
----------------
5chmidti wrote:

I don't *think* that these are needed/should be here, thoughts?

https://github.com/llvm/llvm-project/pull/89553


More information about the llvm-commits mailing list