[PATCH] D33790: [ThinLTO] Assign ValueId only once per GUID when writing combined summary

[Peter Collingbourne via Phabricator via llvm-commits]

pcc added a comment.

Have you benchmarked this change? It seems that it wouldn't make a difference for the WriteIndexesThinBackend because it always passes a ModuleToSummariesForIndex map.


https://reviews.llvm.org/D33790