[PATCH] D30365: ELF ICF: Merge only functions.

[Rui Ueyama via llvm-commits]

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...
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20170226/3cdb6e91/attachment.html>