[clang][modules] Serialize `Module::DefinitionLoc`
authorJan Svoboda <jan_svoboda@apple.com>
Thu, 6 Jul 2023 16:18:03 +0000 (18:18 +0200)
committerJan Svoboda <jan_svoboda@apple.com>
Mon, 17 Jul 2023 20:50:23 +0000 (13:50 -0700)
commitabcf7ce45794839a473a236d55d163016cde5ba6
treea315848bf1e61645b864f1027f13fe3c6f8a9f95
parentcbc4bbb85c729f34bf0cba84ccd2e116af1454f5
[clang][modules] Serialize `Module::DefinitionLoc`

This is a prep patch for avoiding the quadratic number of calls to `HeaderSearch::lookupModule()` in `ASTReader` for each (transitively) loaded PCM file. (Specifically in the context of `clang-scan-deps`).

This patch explicitly serializes `Module::DefinitionLoc` so that we can stop relying on it being filled by the module map parser. This change also required change to the module map parser, where we used the absence of `DefinitionLoc` to determine whether a file came from a PCM file. We also need to make sure we consider the "containing" module map affecting when writing a PCM, so that it's not stripped during serialization, which ensures `DefinitionLoc` still ends up pointing to the correct offset. This is intended to be a NFC change.

Reviewed By: benlangmuir

Differential Revision: https://reviews.llvm.org/D150292
clang/include/clang/Serialization/ASTBitCodes.h
clang/lib/Lex/ModuleMap.cpp
clang/lib/Serialization/ASTReader.cpp
clang/lib/Serialization/ASTWriter.cpp