[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