[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