From: Douglas Gregor <dgregor@apple.com>
Date: Fri, 9 Sep 2011 21:09:37 +0000 (+0000)
Subject: Mark the translation unit as having lexical/visible storage only when we actually... 
X-Git-Url: https://granicus.if.org/sourcecode?a=commitdiff_plain;h=5db255e02fde04cbfec65e4596a217591933a76b;p=clang

Mark the translation unit as having lexical/visible storage only when we actually have data for that lexical or visible storage

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

diff --git a/lib/Serialization/ASTReader.cpp b/lib/Serialization/ASTReader.cpp
index e574e25f5b..55dd213286 100644
--- a/lib/Serialization/ASTReader.cpp
+++ b/lib/Serialization/ASTReader.cpp
@@ -2817,6 +2817,11 @@ void ASTReader::InitializeContext(ASTContext &Ctx) {
       DeclContextInfo Info = DCU->second;
       (*M)->DeclContextInfos.erase(DCU);
       (*M)->DeclContextInfos[TU] = Info;
+      
+      if (Info.NumLexicalDecls)
+        TU->setHasExternalLexicalStorage();
+      if (Info.NameLookupTableData)
+        TU->setHasExternalVisibleStorage();
     }
   }
   
@@ -2828,10 +2833,6 @@ void ASTReader::InitializeContext(ASTContext &Ctx) {
   // if there are any.
   loadDeclUpdateRecords(PREDEF_DECL_TRANSLATION_UNIT_ID, TU);
   
-  // Note that the translation unit has external lexical and visible storage.
-  TU->setHasExternalLexicalStorage(true);
-  TU->setHasExternalVisibleStorage(true);
-
   // FIXME: Find a better way to deal with collisions between these
   // built-in types. Right now, we just ignore the problem.