diff options
| author | Jan Svoboda <jan_svoboda@apple.com> | 2023-07-06 18:18:03 +0200 |
|---|---|---|
| committer | Jan Svoboda <jan_svoboda@apple.com> | 2023-07-17 13:50:23 -0700 |
| commit | abcf7ce45794839a473a236d55d163016cde5ba6 (patch) | |
| tree | a315848bf1e61645b864f1027f13fe3c6f8a9f95 /llvm/docs/CommandGuide | |
| parent | cbc4bbb85c729f34bf0cba84ccd2e116af1454f5 (diff) | |
| download | llvm-abcf7ce45794839a473a236d55d163016cde5ba6.zip llvm-abcf7ce45794839a473a236d55d163016cde5ba6.tar.gz llvm-abcf7ce45794839a473a236d55d163016cde5ba6.tar.bz2 | |
[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
Diffstat (limited to 'llvm/docs/CommandGuide')
0 files changed, 0 insertions, 0 deletions