HandleVTable assert is far too intrusive to stay.

git-svn-id: http://root.cern.ch/svn/root/trunk@47416 27541ba8-7e3a-0410-8455-c3a389f83636

lib/Interpreter/DeclCollector.cpp

@@ -25,7 +25,7 @@ namespace cling {
   }
 
   void DeclCollector::HandleInterestingDecl(DeclGroupRef DGR) {
-    assert("Not implemented yet!");
+    assert(0 && "Not implemented yet!");
   }
 
   // Does more than we want:
@@ -39,7 +39,12 @@ namespace cling {
   }
 
   void DeclCollector::HandleVTable(CXXRecordDecl* RD, bool DefinitionRequired) {
-    assert(0 && "Not implemented yet!");
+    // Intentional no-op. It comes through Sema::DefineUsedVTables, which
+    // comes either Sema::ActOnEndOfTranslationUnit or while instantiating a
+    // template. In our case we will do it on transaction commit, without 
+    // keeping track of used vtables, because we have cases where we bypass the
+    // clang/AST and directly ask the module so that we have to generate 
+    // everything without extra smartness.
   }
 
   void DeclCollector::CompleteTentativeDefinition(VarDecl* VD) {