diff options
Diffstat (limited to 'lib')
-rw-r--r-- | lib/Frontend/ASTUnit.cpp | 2 | ||||
-rw-r--r-- | lib/Serialization/ASTReader.cpp | 57 | ||||
-rw-r--r-- | lib/Serialization/ASTWriter.cpp | 13 |
3 files changed, 44 insertions, 28 deletions
diff --git a/lib/Frontend/ASTUnit.cpp b/lib/Frontend/ASTUnit.cpp index cd4627bc99..c2901c1827 100644 --- a/lib/Frontend/ASTUnit.cpp +++ b/lib/Frontend/ASTUnit.cpp @@ -2381,7 +2381,7 @@ void ASTUnit::TranslateStoredDiagnostics( SmallVector<StoredDiagnostic, 4> Result; Result.reserve(Diags.size()); assert(MMan && "Don't have a module manager"); - serialization::Module *Mod = MMan->Modules.lookup(ModName); + serialization::Module *Mod = MMan->ModuleMgr.lookup(ModName); assert(Mod && "Don't have preamble module"); SLocRemap &Remap = Mod->SLocRemap; for (unsigned I = 0, N = Diags.size(); I != N; ++I) { diff --git a/lib/Serialization/ASTReader.cpp b/lib/Serialization/ASTReader.cpp index 14ba59c1e0..3994fe6b7b 100644 --- a/lib/Serialization/ASTReader.cpp +++ b/lib/Serialization/ASTReader.cpp @@ -2223,7 +2223,7 @@ ASTReader::ReadASTBlock(Module &F) { uint32_t Offset = io::ReadUnalignedLE32(Data); uint16_t Len = io::ReadUnalignedLE16(Data); StringRef Name = StringRef((const char*)Data, Len); - Module *OM = Modules.lookup(Name); + Module *OM = ModuleMgr.lookup(Name); if (!OM) { Error("SourceLocation remap refers to unknown module"); return Failure; @@ -2236,7 +2236,6 @@ ASTReader::ReadASTBlock(Module &F) { break; } - case SOURCE_MANAGER_LINE_TABLE: if (ParseLineTable(F, Record)) return Failure; @@ -2637,19 +2636,7 @@ ASTReader::ASTReadResult ASTReader::ReadAST(const std::string &FileName, ASTReader::ASTReadResult ASTReader::ReadASTCore(StringRef FileName, ModuleKind Type) { - Module *Prev = !ModuleMgr.size() ? 0 : &ModuleMgr.getLastModule(); - ModuleMgr.addModule(Type); - Module &F = ModuleMgr.getLastModule(); - if (Prev) - Prev->NextInSource = &F; - else - FirstInSource = &F; - F.Loaders.push_back(Prev); - - Modules[FileName.str()] = &F; - - // Set the AST file name. - F.FileName = FileName; + Module &F = ModuleMgr.addModule(FileName, Type); if (FileName != "-") { CurrentDir = llvm::sys::path::parent_path(FileName); @@ -4388,7 +4375,7 @@ void ASTReader::InitializeSema(Sema &S) { SemaObj->StdBadAlloc = SemaDeclRefs[1]; } - for (Module *F = FirstInSource; F; F = F->NextInSource) { + for (Module *F = &ModuleMgr.getPrimaryModule(); F; F = F->NextInSource) { // If there are @selector references added them to its pool. This is for // implementation of -Wselector. @@ -5275,7 +5262,7 @@ ASTReader::ASTReader(Preprocessor &PP, ASTContext *Context, : Listener(new PCHValidator(PP, *this)), DeserializationListener(0), SourceMgr(PP.getSourceManager()), FileMgr(PP.getFileManager()), Diags(PP.getDiagnostics()), SemaObj(0), PP(&PP), Context(Context), - Consumer(0), FirstInSource(0), RelocatablePCH(false), isysroot(isysroot), + Consumer(0), RelocatablePCH(false), isysroot(isysroot), DisableValidation(DisableValidation), DisableStatCache(DisableStatCache), NumStatHits(0), NumStatMisses(0), NumSLocEntriesRead(0), TotalNumSLocEntries(0), @@ -5294,7 +5281,7 @@ ASTReader::ASTReader(SourceManager &SourceMgr, FileManager &FileMgr, Diagnostic &Diags, StringRef isysroot, bool DisableValidation, bool DisableStatCache) : DeserializationListener(0), SourceMgr(SourceMgr), FileMgr(FileMgr), - Diags(Diags), SemaObj(0), PP(0), Context(0), Consumer(0), FirstInSource(0), + Diags(Diags), SemaObj(0), PP(0), Context(0), Consumer(0), RelocatablePCH(false), isysroot(isysroot), DisableValidation(DisableValidation), DisableStatCache(DisableStatCache), NumStatHits(0), NumStatMisses(0), NumSLocEntriesRead(0), @@ -5352,3 +5339,37 @@ Module::~Module() { delete static_cast<HeaderFileInfoLookupTable *>(HeaderFileInfoTable); delete static_cast<ASTSelectorLookupTable *>(SelectorLookupTable); } + +/// \brief Creates a new module and adds it to the list of known modules +Module &ModuleManager::addModule(StringRef FileName, ModuleKind Type) { + Module *Prev = !size() ? 0 : &getLastModule(); + Module *Current = new Module(Type); + + Current->FileName = FileName.str(); + + Chain.push_back(Current); + Modules[FileName.str()] = Current; + + if (Prev) + Prev->NextInSource = Current; + Current->Loaders.push_back(Prev); + + return *Current; +} + +/// \brief Exports the list of loaded modules with their corresponding names +void ModuleManager::exportLookup(SmallVector<ModuleOffset, 16> &Target) { + Target.reserve(size()); + for (llvm::StringMap<Module*>::const_iterator + I = Modules.begin(), E = Modules.end(); + I != E; ++I) { + Target.push_back(ModuleOffset(I->getValue()->SLocEntryBaseOffset, + I->getKey())); + } + std::sort(Target.begin(), Target.end()); +} + +ModuleManager::~ModuleManager() { + for (unsigned i = 0, e = Chain.size(); i != e; ++i) + delete Chain[e - i - 1]; +} diff --git a/lib/Serialization/ASTWriter.cpp b/lib/Serialization/ASTWriter.cpp index c94b96d18c..14824a0810 100644 --- a/lib/Serialization/ASTWriter.cpp +++ b/lib/Serialization/ASTWriter.cpp @@ -969,7 +969,8 @@ void ASTWriter::WriteMetadata(ASTContext &Context, StringRef isysroot, Record.push_back(CLANG_VERSION_MINOR); Record.push_back(!isysroot.empty()); // FIXME: This writes the absolute path for chained headers. - const std::string &BlobStr = Chain ? Chain->getFileName() : Target.getTriple().getTriple(); + const std::string &BlobStr = + Chain ? Chain->getFileName() : Target.getTriple().getTriple(); Stream.EmitRecordWithBlob(MetaAbbrevCode, Record, BlobStr); // Original file name and file ID @@ -1548,14 +1549,8 @@ void ASTWriter::WriteSourceManagerBlock(SourceManager &SourceMgr, // Sorted by offset. typedef std::pair<uint32_t, StringRef> ModuleOffset; SmallVector<ModuleOffset, 16> Modules; - Modules.reserve(Chain->Modules.size()); - for (llvm::StringMap<Module*>::const_iterator - I = Chain->Modules.begin(), E = Chain->Modules.end(); - I != E; ++I) { - Modules.push_back(ModuleOffset(I->getValue()->SLocEntryBaseOffset, - I->getKey())); - } - std::sort(Modules.begin(), Modules.end()); + + Chain->ModuleMgr.exportLookup(Modules); Abbrev = new BitCodeAbbrev(); Abbrev->Add(BitCodeAbbrevOp(SOURCE_LOCATION_MAP)); |