From: Douglas Gregor <dgregor@apple.com>
Date: Thu, 17 Feb 2011 18:59:06 +0000 (+0000)
Subject: The internal -fdump-record-layouts flag already dumps the layout when it was computed... 
X-Git-Url: https://granicus.if.org/sourcecode?a=commitdiff_plain;h=d9015eee474115c8ecf616a02f3ed30be2b91203;p=clang

The internal -fdump-record-layouts flag already dumps the layout when it was computed; no need to do so again at the end of the translation unit

git-svn-id: https://llvm.org/svn/llvm-project/cfe/trunk@125760 91177308-0d34-0410-b5e6-96231b3b80d8
---

diff --git a/lib/Parse/ParseAST.cpp b/lib/Parse/ParseAST.cpp
index d02787941b..edb1675b99 100644
--- a/lib/Parse/ParseAST.cpp
+++ b/lib/Parse/ParseAST.cpp
@@ -25,26 +25,6 @@
 
 using namespace clang;
 
-static void DumpRecordLayouts(ASTContext &C) {
-  for (ASTContext::type_iterator I = C.types_begin(), E = C.types_end();
-       I != E; ++I) {
-    const RecordType *RT = dyn_cast<RecordType>(*I);
-    if (!RT)
-      continue;
-
-    const CXXRecordDecl *RD = dyn_cast<CXXRecordDecl>(RT->getDecl());
-    if (!RD || RD->isImplicit() || RD->isDependentType() ||
-        RD->isInvalidDecl() || !RD->getDefinition())
-      continue;
-
-    // FIXME: Do we really need to hard code this?
-    if (RD->getQualifiedNameAsString() == "__va_list_tag")
-      continue;
-
-    C.DumpRecordLayout(RD, llvm::errs());
-  }
-}
-
 //===----------------------------------------------------------------------===//
 // Public interface to the file
 //===----------------------------------------------------------------------===//
@@ -97,10 +77,6 @@ void clang::ParseAST(Sema &S, bool PrintStats) {
        E = S.WeakTopLevelDecls().end(); I != E; ++I)
     Consumer->HandleTopLevelDecl(DeclGroupRef(*I));
   
-  // Dump record layouts, if requested.
-  if (S.getLangOptions().DumpRecordLayouts)
-    DumpRecordLayouts(S.getASTContext());
-  
   Consumer->HandleTranslationUnit(S.getASTContext());
   
   if (PrintStats) {