[PATCH] D44397: [ThinLTO] Clear dllimport when setting dso_local

Rafael Avila de Espindola via llvm-commits llvm-commits at lists.llvm.org
Tue Mar 13 08:10:36 PDT 2018


Teresa Johnson via Phabricator via llvm-commits
<llvm-commits at lists.llvm.org> writes:

> tejohnson accepted this revision.
> tejohnson added a comment.
> This revision is now accepted and ready to land.
>
> LGTM, but what about the regular LTO case (i.e. we set this directly on the GV in LTO.cpp, instead of on the summary and then propagating in the backends here for ThinLTO)?

The regular lib/Linker code already handles this.

Cheers,
Rafael


More information about the llvm-commits mailing list