================
@@ -8546,10 +8546,12 @@ class Sema final : public SemaBase {
                                 bool Diagnose = true);
   FunctionDecl *FindUsualDeallocationFunction(SourceLocation StartLoc,
                                               ImplicitDeallocationParameters,
-                                              DeclarationName Name);
-  FunctionDecl *FindDeallocationFunctionForDestructor(SourceLocation StartLoc,
-                                                      CXXRecordDecl *RD,
-                                                      bool Diagnose = true);
+                                              DeclarationName Name,
+                                              bool Diagnose = true);
+  FunctionDecl *
+  FindDeallocationFunctionForDestructor(SourceLocation StartLoc,
+                                        CXXRecordDecl *RD, bool Diagnose = 
true,
+                                        bool LookForGlobal = false);
----------------
Fznamznon wrote:

Mmm, I would say it has been addressed, see

https://github.com/llvm/llvm-project/blob/ff4c4997ee72f4fda2d9939faefe8ef262d294a8/clang/include/clang/Sema/Sema.h#L8562

Did you also mean to not add defaulted arguments to 
`FindUsualDeallocationFunction`? It has only one. Should I push a change to 
remove it too?

https://github.com/llvm/llvm-project/pull/139566
_______________________________________________
cfe-commits mailing list
[email protected]
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Reply via email to