summaryrefslogtreecommitdiffstats
path: root/clang
diff options
context:
space:
mode:
authorEli Friedman <eli.friedman@gmail.com>2011-08-02 18:05:30 +0000
committerEli Friedman <eli.friedman@gmail.com>2011-08-02 18:05:30 +0000
commitb23533db139e0398a8db884c6e2c1f7d4bf7c720 (patch)
tree3ac72755a7dc8811c741c95844320cd03a96d862 /clang
parentbe10f9853c638c609137c1f5a3256acb742b2eae (diff)
downloadbcm5719-llvm-b23533db139e0398a8db884c6e2c1f7d4bf7c720.tar.gz
bcm5719-llvm-b23533db139e0398a8db884c6e2c1f7d4bf7c720.zip
PR10566: Make sure codegen for deleting an pointer to an incomplete type actually works.
llvm-svn: 136703
Diffstat (limited to 'clang')
-rw-r--r--clang/lib/CodeGen/CGExprCXX.cpp2
-rw-r--r--clang/test/CodeGenCXX/delete.cpp8
2 files changed, 6 insertions, 4 deletions
diff --git a/clang/lib/CodeGen/CGExprCXX.cpp b/clang/lib/CodeGen/CGExprCXX.cpp
index 7967c822b36..738aac2595e 100644
--- a/clang/lib/CodeGen/CGExprCXX.cpp
+++ b/clang/lib/CodeGen/CGExprCXX.cpp
@@ -1206,7 +1206,7 @@ static void EmitObjectDelete(CodeGenFunction &CGF,
const CXXDestructorDecl *Dtor = 0;
if (const RecordType *RT = ElementType->getAs<RecordType>()) {
CXXRecordDecl *RD = cast<CXXRecordDecl>(RT->getDecl());
- if (!RD->hasTrivialDestructor()) {
+ if (RD->hasDefinition() && !RD->hasTrivialDestructor()) {
Dtor = RD->getDestructor();
if (Dtor->isVirtual()) {
diff --git a/clang/test/CodeGenCXX/delete.cpp b/clang/test/CodeGenCXX/delete.cpp
index dbef6a0ebcd..8ad76f8532b 100644
--- a/clang/test/CodeGenCXX/delete.cpp
+++ b/clang/test/CodeGenCXX/delete.cpp
@@ -125,9 +125,11 @@ namespace test4 {
namespace test5 {
struct Incomplete;
- // CHECK: define void @_ZN5test523array_delete_incompleteEPNS_10IncompleteE
- void array_delete_incomplete(Incomplete *p) {
+ // CHECK: define void @_ZN5test523array_delete_incompleteEPNS_10IncompleteES1_
+ void array_delete_incomplete(Incomplete *p1, Incomplete *p2) {
+ // CHECK: call void @_ZdlPv
+ delete p1;
// CHECK: call void @_ZdaPv
- delete [] p;
+ delete [] p2;
}
}
OpenPOWER on IntegriCloud