================
@@ -356,6 +364,14 @@ class TrivialFunctionAnalysisVisitor
     return TrivialFunctionAnalysis::isTrivialImpl(Callee, Cache);
   }
 
+  bool VisitCXXDefaultArgExpr(const CXXDefaultArgExpr *E) {
----------------
haoNoQ wrote:

In theory this may need to be cached because it may cause default argument 
expressions to be visited multiple times, i.e. every time a function is called 
with the given argument omitted. And these expressions can get quite 
complicated.

I haven't ever seen it matter though.

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

Reply via email to