[PATCH] D39302: Don't internalize dllexport functions.

Filipe Cabecinhas via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Thu Oct 26 03:42:44 PDT 2017


filcab added a comment.

In https://reviews.llvm.org/D39302#906950, @pcc wrote:

> It looks like this will only work for full LTO, and not ThinLTO.
>
> How does your linker decide which symbols are exported from regular object files? Should there instead be a mechanism to get that information from the `lto::InputFile`?


Regular object files (ELF-like) have a `.linker_cmd` section which has a few possible commands. One of them is "This symbol is `dllexported`.
I'll look into the thinLTO case. Is the test in an appropriate place, btw?

Thank you,

Filipe


https://reviews.llvm.org/D39302





More information about the llvm-commits mailing list