Teach LTOModule to emit linker flags for dllexported symbols, plus interface cleanup.
authorPeter Collingbourne <peter@pcc.me.uk>
Mon, 29 Jun 2015 22:04:09 +0000 (22:04 +0000)
committerPeter Collingbourne <peter@pcc.me.uk>
Mon, 29 Jun 2015 22:04:09 +0000 (22:04 +0000)
commita6367d913614539a81f2c1cb0f327044810d9956
tree16944f01072c0ba8a62d085099a19287868feaff
parentb37095978832fea1f90fab8317b79e6cf3593928
Teach LTOModule to emit linker flags for dllexported symbols, plus interface cleanup.

This change unifies how LTOModule and the backend obtain linker flags
for globals: via a new TargetLoweringObjectFile member function named
emitLinkerFlagsForGlobal. A new function LTOModule::getLinkerOpts() returns
the list of linker flags as a single concatenated string.

This change affects the C libLTO API: the function lto_module_get_*deplibs now
exposes an empty list, and lto_module_get_*linkeropts exposes a single element
which combines the contents of all observed flags. libLTO should never have
tried to parse the linker flags; it is the linker's job to do so. Because
linkers will need to be able to parse flags in regular object files, it
makes little sense for libLTO to have a redundant mechanism for doing so.

The new API is compatible with the old one. It is valid for a user to specify
multiple linker flags in a single pragma directive like this:

 #pragma comment(linker, "/defaultlib:foo /defaultlib:bar")

The previous implementation would not have exposed
either flag via lto_module_get_*deplibs (as the test in
TargetLoweringObjectFileCOFF::getDepLibFromLinkerOpt was case sensitive)
and would have exposed "/defaultlib:foo /defaultlib:bar" as a single flag via
lto_module_get_*linkeropts. This may have been a bug in the implementation,
but it does give us a chance to fix the interface.

Differential Revision: http://reviews.llvm.org/D10548

git-svn-id: https://llvm.org/svn/llvm-project/llvm/trunk@241010 91177308-0d34-0410-b5e6-96231b3b80d8
include/llvm-c/lto.h
include/llvm/CodeGen/TargetLoweringObjectFileImpl.h
include/llvm/LTO/LTOModule.h
include/llvm/Target/TargetLoweringObjectFile.h
lib/CodeGen/TargetLoweringObjectFileImpl.cpp
lib/LTO/LTOModule.cpp
lib/Target/X86/X86AsmPrinter.cpp
lib/Target/X86/X86AsmPrinter.h
test/CodeGen/X86/dllexport-x86_64.ll
test/CodeGen/X86/dllexport.ll
tools/lto/lto.cpp