diff options
| -rw-r--r-- | lld/ELF/InputFiles.cpp | 9 | ||||
| -rw-r--r-- | lld/test/ELF/debug-gnu-pubnames.s | 10 |
2 files changed, 19 insertions, 0 deletions
diff --git a/lld/ELF/InputFiles.cpp b/lld/ELF/InputFiles.cpp index 3b92ff16fb3..c796fdadd82 100644 --- a/lld/ELF/InputFiles.cpp +++ b/lld/ELF/InputFiles.cpp @@ -461,6 +461,15 @@ elf::ObjectFile<ELFT>::createInputSection(const Elf_Shdr &Sec, if (Config->Strip != StripPolicy::None && Name.startswith(".debug")) return &InputSection::Discarded; + // If -gdb-index is given, LLD creates .gdb_index section, and that + // section serves the same purpose as .debug_gnu_pub{names,types} sections. + // If that's the case, we want to eliminate .debug_gnu_pub{names,types} + // because they are redundant and can waste large amount of disk space + // (for example, they are about 400 MiB in total for a clang debug build.) + if (Config->GdbIndex && + (Name == ".debug_gnu_pubnames" || Name == ".debug_gnu_pubtypes")) + return &InputSection::Discarded; + // The linkonce feature is a sort of proto-comdat. Some glibc i386 object // files contain definitions of symbol "__x86.get_pc_thunk.bx" in linkonce // sections. Drop those sections to avoid duplicate symbol errors. diff --git a/lld/test/ELF/debug-gnu-pubnames.s b/lld/test/ELF/debug-gnu-pubnames.s new file mode 100644 index 00000000000..0a8693c97eb --- /dev/null +++ b/lld/test/ELF/debug-gnu-pubnames.s @@ -0,0 +1,10 @@ +# REQUIRES: x86 +# RUN: ld.lld -e main %p/Inputs/gdb-index-a.elf %p/Inputs/gdb-index-b.elf -o %t1.exe +# RUN: llvm-readobj -sections %t1.exe | FileCheck -check-prefix=CHECK1 %s +# CHECK1: Name: .debug_gnu_pubnames +# CHECK1: Name: .debug_gnu_pubtypes + +# RUN: ld.lld -gdb-index -e main %p/Inputs/gdb-index-a.elf %p/Inputs/gdb-index-b.elf -o %t2.exe +# RUN: llvm-readobj -sections %t2.exe | FileCheck -check-prefix=CHECK2 %s +# CHECK2-NOT: Name: .debug_gnu_pubnames +# CHECK2-NOT: Name: .debug_gnu_pubtypes |
