commit 7228117108b2c3b622a9607255bbb2be4e8bc6f5
author Anders Carlsson <andersca@mac.com> Tue Jan 26 17:36:47 2010 +0000
committer Anders Carlsson <andersca@mac.com> Tue Jan 26 17:36:47 2010 +0000
tree f8bab690cfe095c9fa2d5aa057f1b836f8dfc2ee
parent a6046ab82bd8117dbc055fbbc0621364ee715a39

Change getUnique to return a GlobalDecl. Fixes PR6147.

llvm-svn: 94554

clang/lib/CodeGen/CGVtable.cpp

diff --git a/clang/lib/CodeGen/CGVtable.cpp b/clang/lib/CodeGen/CGVtable.cpp
index 3d05b39..39962e01 100644
--- a/clang/lib/CodeGen/CGVtable.cpp
+++ b/clang/lib/CodeGen/CGVtable.cpp
@@ -415,8 +415,17 @@
     }
     return U;
   }
-  const CXXMethodDecl *getUnique(GlobalDecl GD) {
-    return getUnique(cast<CXXMethodDecl>(GD.getDecl()));
+
+  GlobalDecl getUnique(GlobalDecl GD) {
+    const CXXMethodDecl *Unique = getUnique(cast<CXXMethodDecl>(GD.getDecl()));
+    
+    if (const CXXConstructorDecl *CD = dyn_cast<CXXConstructorDecl>(Unique))
+      return GlobalDecl(CD, GD.getCtorType());
+    
+    if (const CXXDestructorDecl *DD = dyn_cast<CXXDestructorDecl>(Unique))
+      return GlobalDecl(DD, GD.getDtorType());
+    
+    return Unique;
   }
 
   /// getPureVirtualFn - Return the __cxa_pure_virtual function.
@@ -627,13 +636,15 @@
 
     VCallOffset[GD] = Offset/8;
     if (MorallyVirtual) {
-      const CXXMethodDecl *UMD = getUnique(GD);
+      GlobalDecl UGD = getUnique(GD);
+      const CXXMethodDecl *UMD = cast<CXXMethodDecl>(UGD.getDecl());
+  
       assert(UMD && "final overrider not found");
 
       Index_t &idx = VCall[UMD];
       // Allocate the first one, after that, we reuse the previous one.
       if (idx == 0) {
-        VCallOffsetForVCall[UMD] = Offset/8;
+        VCallOffsetForVCall[UGD] = Offset/8;
         NonVirtualOffset[UMD] = -CurrentVBaseOffset/8 + Offset/8;
         idx = VCalls.size()+1;
         VCalls.push_back(Offset/8 - CurrentVBaseOffset/8);
@@ -1069,9 +1080,10 @@
 
     Methods.OverrideMethod(OGD, GD);
 
-    const CXXMethodDecl *UMD = getUnique(GD);
-    assert(UMD && "unique overrider not found");
-    assert(UMD == getUnique(OGD) && "unique overrider not unique");
+    GlobalDecl UGD = getUnique(GD);
+    const CXXMethodDecl *UMD = cast<CXXMethodDecl>(UGD.getDecl());
+    assert(UGD.getDecl() && "unique overrider not found");
+    assert(UGD == getUnique(OGD) && "unique overrider not unique");
 
     ThisAdjustments.erase(Index);
     if (MorallyVirtual || VCall.count(UMD)) {
@@ -1087,7 +1099,7 @@
                   (int)VCalls[idx-1], MostDerivedClass->getNameAsCString()));
       } else {
         VCallOffset[GD] = VCallOffset[OGD];
-        VCalls[idx-1] = -VCallOffsetForVCall[UMD] + OverrideOffset/8;
+        VCalls[idx-1] = -VCallOffsetForVCall[UGD] + OverrideOffset/8;
         D1(printf("  vcall patch for %s at %d with delta %d most derived %s\n",
                   MD->getNameAsString().c_str(), (int)-idx-3,
                   (int)VCalls[idx-1], MostDerivedClass->getNameAsCString()));