diff options
author | Peter Collingbourne <peter@pcc.me.uk> | 2015-06-29 22:04:09 +0000 |
---|---|---|
committer | Peter Collingbourne <peter@pcc.me.uk> | 2015-06-29 22:04:09 +0000 |
commit | aef3659e1888753ffe03510c513e1a2d9101a795 (patch) | |
tree | 90628139a0ed05a1976cc268cde21abd210acfc0 /llvm/lib/Support/LockFileManager.cpp | |
parent | 3c7828e36ebc615904627561eec6761cca2cbab2 (diff) | |
download | llvm-aef3659e1888753ffe03510c513e1a2d9101a795.zip llvm-aef3659e1888753ffe03510c513e1a2d9101a795.tar.gz llvm-aef3659e1888753ffe03510c513e1a2d9101a795.tar.bz2 |
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
llvm-svn: 241010
Diffstat (limited to 'llvm/lib/Support/LockFileManager.cpp')
0 files changed, 0 insertions, 0 deletions