diff options
author | Chandler Carruth <chandlerc@gmail.com> | 2010-08-08 07:04:00 +0000 |
---|---|---|
committer | Chandler Carruth <chandlerc@gmail.com> | 2010-08-08 07:04:00 +0000 |
commit | 09556fd1fb576144e2beda023bf3386f2292243b (patch) | |
tree | aa4c7237b48ba6c71009fe24c2f8d8c4477f609d | |
parent | a570260357155454465815c292a14a1c43d9213a (diff) |
Fix a crash on template delete operators.
git-svn-id: https://llvm.org/svn/llvm-project/cfe/trunk@110542 91177308-0d34-0410-b5e6-96231b3b80d8
-rw-r--r-- | lib/Sema/SemaExprCXX.cpp | 10 | ||||
-rw-r--r-- | test/SemaCXX/new-delete.cpp | 12 |
2 files changed, 20 insertions, 2 deletions
diff --git a/lib/Sema/SemaExprCXX.cpp b/lib/Sema/SemaExprCXX.cpp index b1d6015514..7bebce16e1 100644 --- a/lib/Sema/SemaExprCXX.cpp +++ b/lib/Sema/SemaExprCXX.cpp @@ -1301,8 +1301,14 @@ bool Sema::FindDeallocationFunction(SourceLocation StartLoc, CXXRecordDecl *RD, llvm::SmallVector<DeclAccessPair,4> Matches; for (LookupResult::iterator F = Found.begin(), FEnd = Found.end(); F != FEnd; ++F) { - CXXMethodDecl *Delete = cast<CXXMethodDecl>((*F)->getUnderlyingDecl()); - if (Delete->isUsualDeallocationFunction()) + NamedDecl *ND = (*F)->getUnderlyingDecl(); + + // Ignore template operator delete members from the check for a usual + // deallocation function. + if (isa<FunctionTemplateDecl>(ND)) + continue; + + if (cast<CXXMethodDecl>(ND)->isUsualDeallocationFunction()) Matches.push_back(F.getPair()); } diff --git a/test/SemaCXX/new-delete.cpp b/test/SemaCXX/new-delete.cpp index 816b808346..b127e667c5 100644 --- a/test/SemaCXX/new-delete.cpp +++ b/test/SemaCXX/new-delete.cpp @@ -332,3 +332,15 @@ namespace PR7810 { static void operator delete(void *volatile); }; } + +// Don't crash on template delete operators +namespace TemplateDestructors { + struct S { + virtual ~S() {} + + void* operator new(const size_t size); + template<class T> void* operator new(const size_t, const int, T*); + void operator delete(void*, const size_t); + template<class T> void operator delete(void*, const size_t, const int, T*); + }; +} |