[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