[PATCH] D68950: [ThinLTOCodeGenerator] Add support for index-based WPD

Eugene Leviant via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Wed Oct 16 09:32:12 PDT 2019


evgeny777 added a comment.

> Should we add indexed-based WPD into optimized as well?

The problem is where we would get summary with WPD resolutions in such case. The only place where combined index is saved is `linkCombinedIndex` but
it lacks information about exported symbols and it's not possible to devirtualize without it.

Extending linkCombinedIndex, so that --exported-symbol can be used in conjunction with --thinlto-action=thinlink can probably fix the issue.
The next step after that is probably to strip `--exported-symbol` flag from `import` and `promote` actions as combined index would be finalized after thin link is over.


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

https://reviews.llvm.org/D68950





More information about the llvm-commits mailing list