[PATCH] D127777: [ThinLTO][ELF] Add --thinlto-emit-index-files option

Jin Xin Ng via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Wed Jun 22 11:29:02 PDT 2022


Northbadge added a comment.



In D127777#3600008 <https://reviews.llvm.org/D127777#3600008>, @MaskRay wrote:

>> Allows ThinLTO indices to be written to disk on-the-fly/as-part-of “normal” linker execution. Previously thinLTO indices could be written via --thinlto-index-only but that would cause the linker to exit early.
>
> Can you state the motivation for not exiting early?

I've updated the summary, but essentially, for MLGO it lets us collect the ThinLTO index files and any IR or .o files we need in one execution of the linker, rather than 2.

I'm not too sure what you meant with

> What files are needed for --thinlto-emit-index-files?

though. It should require all the files you'd normally pass in to lld to do a ThinLTO link, except this patch enables passing extra options to save the ThinLTO index files.


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D127777/new/

https://reviews.llvm.org/D127777



More information about the llvm-commits mailing list