[PATCH] D40747: [ThinLTO] Enable importing of aliases as copy of aliasee

Teresa Johnson via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Wed Dec 13 08:53:16 PST 2017


tejohnson added a comment.

In https://reviews.llvm.org/D40747#947144, @pcc wrote:

> Now that we basically treat alias summaries in the same way as their aliasees, I wonder whether we can simplify things by removing AliasSummary and summarizing aliases by creating a FunctionSummary or GlobalVariableSummary from the aliasee instead.


I'm not sure we can as there are other requirements for knowing the alias relationship. For example, we can't convert a non-prevailing alias and aliasee to available_externally (see thinLTOResolveWeakForLinkerGUID) - that is in the original module and not related to importing.


https://reviews.llvm.org/D40747





More information about the llvm-commits mailing list