[PATCH] D35081: [ThinLTO] Allow multiple summary entries.

[Mehdi AMINI via Phabricator via cfe-commits]

mehdi_amini added a comment.

In https://reviews.llvm.org/D35081#808189, @tejohnson wrote:

> > Alternatively (and likely preferably from a compile-time point of view to limit the list of import), we could keep a map of GUID->Summary and reuse it before trying to select a new callee.
>
> Yep, that was my third option. I only hesitate because it seems like it might be better to select the prevailing copy when we can, but perhaps it doesn't matter much.


I'm not sure I understand what is your first option? Always selecting the prevailing copy? 
Unless doing a topological order for visiting (which I don't think we could reasonably do), I don't see how you can ensure that:  on your first visit you may see the prevailing not eligible, but it may become eligible on the second visit, at which point you need to do something about the first copy you already selected.


https://reviews.llvm.org/D35081