ThinLTO: Do not import debug info for imported global constants
authorDavid Blaikie <dblaikie@gmail.com>
Wed, 5 Dec 2018 21:42:17 +0000 (21:42 +0000)
committerDavid Blaikie <dblaikie@gmail.com>
Wed, 5 Dec 2018 21:42:17 +0000 (21:42 +0000)
commit01f1d9b5894de7529fc9b0e6aeb6f414be950cb8
tree64034137931c0e2bdabba0e767aa33f7bef74bf6
parent85393b28f99ab23acdf4bfb12694314f9b51c911
ThinLTO: Do not import debug info for imported global constants

It looks like this isn't necessary (in any tests I've done, it results
in the global being described with no location or value in the imported
side - while it's still fully described in the place it's imported from)
& results in significant/pathological debug info growth to home these
location-less global variable descriptions on the import side.

This is a rather pressing/important issue to address - this regressed
executable size for one example I'm looking at by 15%, object size is probably
similar though I haven't measured it, and a 22x increase in the number of CUs
in the cu_index in split DWARF DWP files, creating a similarly large regression
in the time it takes llvm-symbolizer to run on such binaries.

Reviewers: tejohnson, evgeny777

Differential Revision: https://reviews.llvm.org/D55309

llvm-svn: 348416
llvm/lib/Linker/IRMover.cpp
llvm/test/ThinLTO/X86/index-const-prop.ll
llvm/test/ThinLTO/X86/index-const-prop2.ll