[llvm] [PDB] Refactor cache strategy for function symbol lookups (PR #188927)

[via llvm-commits]

================
@@ -355,29 +366,67 @@ SymbolCache::findFunctionSymbolBySectOffset(uint32_t Sect, uint32_t Offset) {
     consumeError(ExpectedModS.takeError());
     return nullptr;
   }
-  CVSymbolArray Syms = ExpectedModS->getSymbolArray();
 
-  // Search for the symbol in this module.
+  // Return empty intervals in AddressToSymbolId from Start to Stop.
+  auto getInsertRanges = [this](uint64_t Start, uint64_t Stop) {
----------------
Nerixyz wrote:

This should usually return one range from `Start` to `Stop`, right? Unless there are two functions that overlap.

Would be great if this could be tested. But I assume that would require yaml2pdb or similar in the unit tests.

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