[clang] [clang] Do not diagnose unused deleted operator delete[] (PR #134357)

llvmbot wrote: @llvm/pr-subscribers-clang Author: Mariya Podchishchaeva (Fznamznon) Changes For vector deleting dtors support we now also search and save operator delete[]. Avoid diagnosing deleted operator delete[] when doing that because vector deleting dtors are only called when delet

[clang] [clang] Do not diagnose unused deleted operator delete[] (PR #134357)

https://github.com/Fznamznon created https://github.com/llvm/llvm-project/pull/134357 For vector deleting dtors support we now also search and save operator delete[]. Avoid diagnosing deleted operator delete[] when doing that because vector deleting dtors are only called when delete[] is prese

[clang] [clang] Do not diagnose unused deleted operator delete[] (PR #134357)

@@ -2878,7 +2878,7 @@ class CXXDestructorDecl : public CXXMethodDecl { static CXXDestructorDecl *CreateDeserialized(ASTContext &C, GlobalDeclID ID); void setOperatorDelete(FunctionDecl *OD, Expr *ThisArg); - void setOperatorArrayDelete(FunctionDecl *OD, Expr *ThisArg); +

[clang] [clang] Do not diagnose unused deleted operator delete[] (PR #134357)

https://github.com/Fznamznon closed https://github.com/llvm/llvm-project/pull/134357 ___ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

[clang] [clang] Do not diagnose unused deleted operator delete[] (PR #134357)

@@ -2878,7 +2878,7 @@ class CXXDestructorDecl : public CXXMethodDecl { static CXXDestructorDecl *CreateDeserialized(ASTContext &C, GlobalDeclID ID); void setOperatorDelete(FunctionDecl *OD, Expr *ThisArg); - void setOperatorArrayDelete(FunctionDecl *OD, Expr *ThisArg); +

[clang] [clang] Do not diagnose unused deleted operator delete[] (PR #134357)

Fznamznon wrote: > LGTM but do we need a release note? No, it is a regression caused by a patch committed a couple of days ago. https://github.com/llvm/llvm-project/pull/134357 ___ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm

[clang] [clang] Do not diagnose unused deleted operator delete[] (PR #134357)

AaronBallman wrote: > > LGTM but do we need a release note? > > No, it is a fix for a regression caused by a patch committed a couple of days > ago. Excellent, thank you for confirming! LG as-is https://github.com/llvm/llvm-project/pull/134357 ___ c

[clang] [clang] Do not diagnose unused deleted operator delete[] (PR #134357)

https://github.com/AaronBallman approved this pull request. LGTM but do we need a release note? https://github.com/llvm/llvm-project/pull/134357 ___ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe

[clang] [clang] Do not diagnose unused deleted operator delete[] (PR #134357)

@@ -2878,7 +2878,7 @@ class CXXDestructorDecl : public CXXMethodDecl { static CXXDestructorDecl *CreateDeserialized(ASTContext &C, GlobalDeclID ID); void setOperatorDelete(FunctionDecl *OD, Expr *ThisArg); - void setOperatorArrayDelete(FunctionDecl *OD, Expr *ThisArg); +

[clang] [clang] Do not diagnose unused deleted operator delete[] (PR #134357)

@@ -2878,7 +2878,7 @@ class CXXDestructorDecl : public CXXMethodDecl { static CXXDestructorDecl *CreateDeserialized(ASTContext &C, GlobalDeclID ID); void setOperatorDelete(FunctionDecl *OD, Expr *ThisArg); - void setOperatorArrayDelete(FunctionDecl *OD, Expr *ThisArg); +

[clang] [clang] Do not diagnose unused deleted operator delete[] (PR #134357)

https://github.com/AaronBallman edited https://github.com/llvm/llvm-project/pull/134357 ___ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits