[clang] [clang][modules] Move `SLocEntry` search into `ASTReader` (PR #66966)

[Ben Langmuir via cfe-commits]

================
@@ -288,10 +288,12 @@ class ModuleFile {
   /// for the entry is SLocEntryOffsetsBase + SLocEntryOffsets[i].
   uint64_t SLocEntryOffsetsBase = 0;
 
-  /// Offsets for all of the source location entries in the
-  /// AST file.
+  /// Stream bit offsets for all of the source location entries in the AST file.
   const uint32_t *SLocEntryOffsets = nullptr;
 
+  /// SLocEntry offsets that have been loaded from the AST file.
+  std::vector<SourceLocation::UIntTy> SLocEntryOffsetLoaded;
----------------
benlangmuir wrote:

I assume this is relatively small memory use in practice?

https://github.com/llvm/llvm-project/pull/66966