[PATCH] D35189: Add GUID-ValueID map and original name to ThinLTO summary.

Peter Collingbourne via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Mon Jul 10 12:30:24 PDT 2017


pcc added a comment.

> the global VST (for the global values we will have in the index) does not.

Not quite, global VSTs include a reference to the symbol name in the strtab.

> But I think infer it from the code in my previous inline comment, it is "we need the name to recompute a GUID for internal symbol promoted to global", correct?

Names of globals are also necessary for symbol resolution.


https://reviews.llvm.org/D35189





More information about the llvm-commits mailing list