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

[Mehdi AMINI via Phabricator via cfe-commits]

mehdi_amini added a comment.

Thanks, very clear :)

I would expect that if we reprocess a GUID we invalidate the previous import of the same GUID. Right now my impression is that the issue is that ` ImportList[ExportModulePath][VI.getGUID()]` is indexed on the importing module. So it'd require for every new import to loop over the ImportList map and delete the GUID on each inner map.

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.


https://reviews.llvm.org/D35081