[PATCH] D19351: ELF: Add initial ThinLTO support.

Rafael EspĂ­ndola via llvm-commits llvm-commits at lists.llvm.org
Fri May 13 05:21:05 PDT 2016


>> If it is possible for lld/gold/LTO to share more code, excellent but:
>>
>> * I would probably not put that in lib/LTO. Lets keep that just for
>> the existing api.
>
>
> Putting the new API in some other lib makes sense to me.
>
>
>
> Can you elaborate on the rational for that? It does not seem obvious to me
> (i.e. that's where I'd go for any new API that is dedicated to LTO).

The existing LTO api has a lot of history and is very different from
what we want to do in the gold plugin and lld. This is similar to
splitting out the IRMover, except sufficiently larger that I think an
independent library that lld, gold (and lib/LTO?) use makes sense and
avoids accidentally adding another use of the old API.

Cheers,
Rafael


More information about the llvm-commits mailing list