[PATCH] D54815: [ThinLTO] Add summary entries for index-based WPD

[Teresa Johnson via Phabricator via llvm-commits]

tejohnson added a comment.

In D54815#1354816 <https://reviews.llvm.org/D54815#1354816>, @davidxl wrote:

> A very high level meta question: can thinLink phase read symbol table from each module? If yes, why duplicate the symtab information into summary?


We do in fact require reading the IRSymtab along with the summary, e.g. the symbol strings are shared. What duplication are you concerned about? I don't think anything added in this patch would be available in the symbol table.


Repository:
  rL LLVM

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

https://reviews.llvm.org/D54815