[cfe-commits] r124201 - in /cfe/trunk/lib/Sema: Sema.cpp SemaDeclCXX.cpp

Anders Carlsson andersca at mac.com
Tue Jan 25 10:08:22 PST 2011


Author: andersca
Date: Tue Jan 25 12:08:22 2011
New Revision: 124201

URL: http://llvm.org/viewvc/llvm-project?rev=124201&view=rev
Log:
Don't insert class templates into the DynamicClasses vector.

Modified:
    cfe/trunk/lib/Sema/Sema.cpp
    cfe/trunk/lib/Sema/SemaDeclCXX.cpp

Modified: cfe/trunk/lib/Sema/Sema.cpp
URL: http://llvm.org/viewvc/llvm-project/cfe/trunk/lib/Sema/Sema.cpp?rev=124201&r1=124200&r2=124201&view=diff
==============================================================================
--- cfe/trunk/lib/Sema/Sema.cpp (original)
+++ cfe/trunk/lib/Sema/Sema.cpp Tue Jan 25 12:08:22 2011
@@ -290,6 +290,8 @@
     // this translation unit, then those vtables are considered "used" and must
     // be emitted.
     for (unsigned I = 0, N = DynamicClasses.size(); I != N; ++I) {
+      assert(!DynamicClasses[I]->isDependentType() &&
+             "Should not see dependent types here!");
       if (const CXXMethodDecl *KeyFunction
           = Context.getKeyFunction(DynamicClasses[I])) {
         const FunctionDecl *Definition = 0;

Modified: cfe/trunk/lib/Sema/SemaDeclCXX.cpp
URL: http://llvm.org/viewvc/llvm-project/cfe/trunk/lib/Sema/SemaDeclCXX.cpp?rev=124201&r1=124200&r2=124201&view=diff
==============================================================================
--- cfe/trunk/lib/Sema/SemaDeclCXX.cpp (original)
+++ cfe/trunk/lib/Sema/SemaDeclCXX.cpp Tue Jan 25 12:08:22 2011
@@ -2746,7 +2746,7 @@
     }
   }
 
-  if (Record->isDynamicClass())
+  if (Record->isDynamicClass() && !Record->isDependentType())
     DynamicClasses.push_back(Record);
 
   if (Record->getIdentifier()) {





More information about the cfe-commits mailing list