[PATCH] D93732: [LLD][COFF] When using PCH.OBJ, ensure func_id records indices are remapped under /DEBUG:GHASH

[Alexandre Ganea via Phabricator via llvm-commits]

aganea added a comment.

Thanks for looking at this!



================
Comment at: lld/COFF/DebugTypes.cpp:1150-1153
 // Fill in a TPI or IPI index map using ghashes. For each source type, use its
 // ghash to lookup its final type index in the PDB, and store that in the map.
 void TpiSource::fillMapFromGHashes(GHashState *g,
                                    SmallVectorImpl<TypeIndex> &mapToFill) {
----------------
rnk wrote:
> I think this is ripe for some follow-on simplifications. I believe `mapToFill` is always just `indexMapStorage`. This routine is really rewriting ghash cell indices to PDB type indices.
Yes indeed, I made that change locally, I could commit a NFC patch after landing this.


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D93732/new/

https://reviews.llvm.org/D93732