[PATCH] D33621: Fix for -wrap linker option and LTO, PR 33145
Rafael Avila de Espindola via llvm-commits
llvm-commits at lists.llvm.org
Mon May 29 10:53:32 PDT 2017
Rui Ueyama via Phabricator <reviews at reviews.llvm.org> writes:
> ruiu added a comment.
>
> Do you really need this much change? I expected something smaller than this. I mean, I thought what we need to do is to just pass a list of symbols which LTO should keep.
There has to be a bit more. If we keep a symbol foo, that foo can still
be inlined across translation units, which is not what one would expect
with a regular link and --wrap.
Cheers,
Rafael
More information about the llvm-commits
mailing list