[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