]> granicus.if.org Git - clang/commitdiff
Fix a trivial typo, add a FIXME to have more test coverage for VTableBuilder
authorTimur Iskhodzhanov <timurrrr@google.com>
Fri, 28 Jun 2013 15:42:28 +0000 (15:42 +0000)
committerTimur Iskhodzhanov <timurrrr@google.com>
Fri, 28 Jun 2013 15:42:28 +0000 (15:42 +0000)
git-svn-id: https://llvm.org/svn/llvm-project/cfe/trunk@185159 91177308-0d34-0410-b5e6-96231b3b80d8

lib/AST/VTableBuilder.cpp

index 74669d565dafd39e9dd9480496584fdb66bc69bf..90087b8271761d01bf80534a08f4bab99fbeca95 100644 (file)
@@ -1914,6 +1914,8 @@ VTableBuilder::LayoutVTablesForVirtualBases(const CXXRecordDecl *RD,
 
 /// dumpLayout - Dump the vtable layout.
 void VTableBuilder::dumpLayout(raw_ostream& Out) {
+  // FIXME: write more tests that actually use the dumpLayout output to prevent
+  // VTableBuilder regressions.
 
   if (isBuildingConstructorVTable()) {
     Out << "Construction vtable for ('";
@@ -2166,7 +2168,7 @@ void VTableBuilder::dumpLayout(raw_ostream& Out) {
         
         // If this function pointer has a return pointer adjustment, dump it.
         if (!Thunk.Return.isEmpty()) {
-          Out << "return adjustment: " << Thunk.This.NonVirtual;
+          Out << "return adjustment: " << Thunk.Return.NonVirtual;
           Out << " non-virtual";
           if (Thunk.Return.VBaseOffsetOffset) {
             Out << ", " << Thunk.Return.VBaseOffsetOffset;