[PATCH] D30365: ELF ICF: Merge only functions.
Rui Ueyama via llvm-commits
llvm-commits at lists.llvm.org
Sun Feb 26 17:21:32 PST 2017
On Sun, Feb 26, 2017 at 5:14 PM, Rafael Avila de Espindola <
rafael.espindola at gmail.com> wrote:
> Rui Ueyama <ruiu at google.com> writes:
> > We can define this as -icf=everything or something (as opposed to the
> > current -icf=all), but do you want that now?
> No, it doesn't have to be now.
> Thinking a bit more about it it is not even clear if we should have
> it. The logic in ICF is not that different and once we have a way of
> representing unnamed_addr in ELF, --icf=all will just mean icf
> executable sections even if they are not know to be safe.
Can you elaborate a bit on why you want to propagate unnamed_addr and how
you use that in the linker?
-------------- next part --------------
An HTML attachment was scrubbed...
More information about the llvm-commits