diff options
author | Richard Smith <richard-llvm@metafoo.co.uk> | 2015-08-24 21:59:32 +0000 |
---|---|---|
committer | Richard Smith <richard-llvm@metafoo.co.uk> | 2015-08-24 21:59:32 +0000 |
commit | d8879c85f68250baa6f2ebc463259201782e8460 (patch) | |
tree | 5620bbc3c8111058e26e69123c027ee38722889b /clang/lib/Serialization/ASTWriter.cpp | |
parent | 20348229812c6592bf72e14cf6f5fa5ff00df157 (diff) | |
download | bcm5719-llvm-d8879c85f68250baa6f2ebc463259201782e8460.tar.gz bcm5719-llvm-d8879c85f68250baa6f2ebc463259201782e8460.zip |
[modules] Remove unnecessary deserialization of fully-external HeaderFileInfos for all files we've seen in this compilation.
llvm-svn: 245881
Diffstat (limited to 'clang/lib/Serialization/ASTWriter.cpp')
-rw-r--r-- | clang/lib/Serialization/ASTWriter.cpp | 6 |
1 files changed, 3 insertions, 3 deletions
diff --git a/clang/lib/Serialization/ASTWriter.cpp b/clang/lib/Serialization/ASTWriter.cpp index 678258eb44f..7058ba3eb9b 100644 --- a/clang/lib/Serialization/ASTWriter.cpp +++ b/clang/lib/Serialization/ASTWriter.cpp @@ -1736,9 +1736,9 @@ void ASTWriter::WriteHeaderSearch(const HeaderSearch &HS) { // changed since it was loaded. Also skip it if it's for a modular header // from a different module; in that case, we rely on the module(s) // containing the header to provide this information. - const HeaderFileInfo *HFI = HS.getExistingFileInfo(File); - if (!HFI || (HFI->External && Chain) || - (HFI->isModuleHeader && !HFI->isCompilingModuleHeader)) + const HeaderFileInfo *HFI = + HS.getExistingFileInfo(File, /*WantExternal*/!Chain); + if (!HFI || (HFI->isModuleHeader && !HFI->isCompilingModuleHeader)) continue; // Massage the file path into an appropriate form. |