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

Rafael Avila de Espindola via llvm-commits llvm-commits at lists.llvm.org
Sun Feb 26 17:14:05 PST 2017


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.

Cheers,
Rafael


More information about the llvm-commits mailing list