[PATCH] D16440: [ThinLTO] Link in only necessary DICompileUnit operands
Teresa Johnson via llvm-commits
llvm-commits at lists.llvm.org
Thu Jan 21 19:12:56 PST 2016
tejohnson created this revision.
tejohnson added reviewers: aprantl, joker.eph.
tejohnson added a subscriber: llvm-commits.
Herald added a subscriber: joker.eph.
As discussed on IRC, for function importing we don't need to link in
the type, enums and many other metadata hanging off the DICompileUnit
for the source module. With this patch, we will only map in the needed
DISubprograms and the DIFile (which is required) and a few simple
non-metadata members.
Do we also need to link in the DIImportedEntity though? We are marking any
DISubprograms reached from them as needed (findNeededSubprograms()).
http://reviews.llvm.org/D16440
Files:
include/llvm/Linker/IRMover.h
lib/Linker/IRMover.cpp
lib/Linker/LinkModules.cpp
test/Linker/thinlto_funcimport_debug.ll
test/Transforms/FunctionImport/funcimport_debug.ll
-------------- next part --------------
A non-text attachment was scrubbed...
Name: D16440.45639.patch
Type: text/x-patch
Size: 6902 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20160122/02c9cdae/attachment.bin>
More information about the llvm-commits
mailing list