]> granicus.if.org Git - clang/commitdiff
PR10566: Make sure codegen for deleting an pointer to an incomplete type actually...
authorEli Friedman <eli.friedman@gmail.com>
Tue, 2 Aug 2011 18:05:30 +0000 (18:05 +0000)
committerEli Friedman <eli.friedman@gmail.com>
Tue, 2 Aug 2011 18:05:30 +0000 (18:05 +0000)
git-svn-id: https://llvm.org/svn/llvm-project/cfe/trunk@136703 91177308-0d34-0410-b5e6-96231b3b80d8

lib/CodeGen/CGExprCXX.cpp
test/CodeGenCXX/delete.cpp

index 7967c822b3628b18d4fe542f3ac5409f75124b53..738aac2595e174729f5cc4ef5b31b149d2c39834 100644 (file)
@@ -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()) {
index dbef6a0ebcd84bf2f37da2df58777e4bd0547bd4..8ad76f8532b43da03ec9bb62f404bf04566e34b5 100644 (file)
@@ -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;
   }
 }