[llvm] r257174 - [ThinLTO] Enable in-place symbol changes for exporting module
Rafael EspĂndola via llvm-commits
llvm-commits at lists.llvm.org
Wed Jan 20 16:27:41 PST 2016
> + if (ImportIndex) {
> + ThinLTOGlobalProcessing ThinLTOProcessing(SrcM, ImportIndex,
> + FunctionsToImport);
> + if (ThinLTOProcessing.run())
> + return true;
> + for (auto *GV : ThinLTOProcessing.getNewExportedValues())
> + ValuesToLink.insert(GV);
> + }
>
This patch is a nice improvement, but it should be possible to move
this completely out of the linker, no? When we have an ImportIndex we
also have a complete list of what to import.
Cheers,
Rafael
More information about the llvm-commits
mailing list