[PATCH] D17212: [ThinLTO] Support for call graph in per-module and combined summary.
Teresa Johnson via llvm-commits
llvm-commits at lists.llvm.org
Fri Feb 26 10:51:16 PST 2016
tejohnson added a comment.
In http://reviews.llvm.org/D17212#362994, @davidxl wrote:
> Is there a need to mark it? Does the following work?
> 1. using summary call graph to do importing decision
> 2. for any function that is imported by another module, mark it as 'exported'
> 3. When processing the exporting module, check any exported function and promote any statics it references.
That does work for correctness, but Mehdi was proposing that functions accessing local values need to meet a higher importing threshold so that we don't cause too many promotions due to over-eager importing.
In terms of serializing the info out for doing any of this with a distributed build, we could do it via an "exported" bit in the combined summary though.
More information about the llvm-commits