From 654e5c7cf802af8b04106bcf6852654fc80144eb Mon Sep 17 00:00:00 2001 From: Anders Carlsson Date: Sat, 14 Nov 2009 03:17:38 +0000 Subject: Diagnose ambiguity of operator delete and operator delete[]. Sebastian, please review. llvm-svn: 88747 --- clang/lib/Sema/SemaExprCXX.cpp | 7 ++++++- 1 file changed, 6 insertions(+), 1 deletion(-) (limited to 'clang/lib/Sema/SemaExprCXX.cpp') diff --git a/clang/lib/Sema/SemaExprCXX.cpp b/clang/lib/Sema/SemaExprCXX.cpp index 964334b20bb..0689c6cce96 100644 --- a/clang/lib/Sema/SemaExprCXX.cpp +++ b/clang/lib/Sema/SemaExprCXX.cpp @@ -851,8 +851,13 @@ Sema::ActOnCXXDelete(SourceLocation StartLoc, bool UseGlobal, // Try to find operator delete/operator delete[] in class scope. LookupResult Found; LookupQualifiedName(Found, Record, DeleteName, LookupOrdinaryName); + + if (Found.isAmbiguous()) { + DiagnoseAmbiguousLookup(Found, DeleteName, StartLoc); + return ExprError(); + } + // FIXME: Diagnose ambiguity properly - assert(!Found.isAmbiguous() && "Ambiguous delete/delete[] not handled"); for (LookupResult::iterator F = Found.begin(), FEnd = Found.end(); F != FEnd; ++F) { if (CXXMethodDecl *Delete = dyn_cast(*F)) -- cgit v1.2.3