================
@@ -98,16 +100,18 @@ void 
UpgradeDurationConversionsCheck::registerMatchers(MatchFinder *Finder) {
   //   `absl::Hours(x)`
   // where `x` is not of a built-in type.
   Finder->addMatcher(
-      traverse(TK_AsIs, implicitCastExpr(
-                            anyOf(hasCastKind(CK_UserDefinedConversion),
-                                  has(implicitCastExpr(
-                                      hasCastKind(CK_UserDefinedConversion)))),
-                            hasParent(callExpr(
-                                callee(functionDecl(
-                                    DurationFactoryFunction(),
-                                    
unless(hasParent(functionTemplateDecl())))),
-                                hasArgument(0, expr().bind("arg")))))
-                            .bind("OuterExpr")),
+      traverse(
+          TK_AsIs,
+          implicitCastExpr(
+              anyOf(
+                  hasCastKind(CK_UserDefinedConversion),
+                  
has(implicitCastExpr(hasCastKind(CK_UserDefinedConversion)))),
+              hasParent(callExpr(
+                  callee(
+                      functionDecl(DurationFactoryFunction(),
+                                   unless(hasParent(functionTemplateDecl())))),
+                  hasArgument(0, ignoringParenImpCasts(expr().bind("arg"))))))
----------------
5chmidti wrote:

This change is not needed, the check does not if implicit nodes are ignored or 
not, so we don't need to do the extra work.

https://github.com/llvm/llvm-project/pull/89553
_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Reply via email to