diff options
author | Haohai Wen <haohai.wen@intel.com> | 2023-04-27 09:32:44 +0800 |
---|---|---|
committer | Haohai Wen <haohai.wen@intel.com> | 2023-04-27 09:42:47 +0800 |
commit | 44363f2ff2736e4edf4a260f442b513ceac661fc (patch) | |
tree | 725adc2feebb1fd92acc236c224f4448da261618 /lld | |
parent | 05551c6582693c2b494ed8c30a46ea5969b507b1 (diff) | |
download | llvm-44363f2ff2736e4edf4a260f442b513ceac661fc.tar.gz |
Always emit symbol table when dwarf section exists in COFF
This also fixes check prefix NO which is pointless in symtab.test
Reviewed By: mstorsjo
Differential Revision: https://reviews.llvm.org/D149235
Diffstat (limited to 'lld')
-rw-r--r-- | lld/COFF/Writer.cpp | 4 | ||||
-rw-r--r-- | lld/test/COFF/symtab.test | 9 |
2 files changed, 11 insertions, 2 deletions
diff --git a/lld/COFF/Writer.cpp b/lld/COFF/Writer.cpp index 603703e65290..645c63f69040 100644 --- a/lld/COFF/Writer.cpp +++ b/lld/COFF/Writer.cpp @@ -1266,7 +1266,9 @@ void Writer::createSymbolAndStringTable() { // solution where discardable sections have long names preserved and // non-discardable sections have their names truncated, to ensure that any // section which is mapped at runtime also has its name mapped at runtime. + bool HasDwarfSection = false; for (OutputSection *sec : ctx.outputSections) { + HasDwarfSection |= sec->name.startswith(".debug_"); if (sec->name.size() <= COFF::NameSize) continue; if ((sec->header.Characteristics & IMAGE_SCN_MEM_DISCARDABLE) == 0) @@ -1279,7 +1281,7 @@ void Writer::createSymbolAndStringTable() { sec->setStringTableOff(addEntryToStringTable(sec->name)); } - if (ctx.config.debugDwarf || ctx.config.debugSymtab) { + if (ctx.config.debugDwarf || ctx.config.debugSymtab || HasDwarfSection) { for (ObjFile *file : ctx.objFileInstances) { for (Symbol *b : file->getSymbols()) { auto *d = dyn_cast_or_null<Defined>(b); diff --git a/lld/test/COFF/symtab.test b/lld/test/COFF/symtab.test index 41419a942d87..f0a160cec790 100644 --- a/lld/test/COFF/symtab.test +++ b/lld/test/COFF/symtab.test @@ -8,6 +8,8 @@ # RUN: lld-link /debug /out:%t.exe /entry:main %t.obj %p/Inputs/std64.lib # RUN: llvm-readobj --symbols %t.exe | FileCheck -check-prefix=NO %s +# RUN: lld-link /debug /merge:.blank=.debug_info /out:%t.exe /entry:main %t.obj %p/Inputs/std64.lib +# RUN: llvm-readobj --symbols %t.exe | FileCheck %s # CHECK: Symbols [ # CHECK-NEXT: Symbol { @@ -84,7 +86,8 @@ # CHECK-NEXT: } # CHECK-NEXT: ] -# NO: Symbols [ +# NO: Symbols [ +# NO-NEXT: ] --- !COFF header: @@ -141,6 +144,10 @@ sections: Characteristics: [ IMAGE_SCN_CNT_INITIALIZED_DATA, IMAGE_SCN_MEM_READ, IMAGE_SCN_MEM_WRITE ] Alignment: 4 SectionData: 48656C6C6F0048656C6C6F20576F726C6400 + - Name: .blank + Characteristics: [ IMAGE_SCN_CNT_INITIALIZED_DATA, IMAGE_SCN_MEM_DISCARDABLE, IMAGE_SCN_MEM_READ ] + Alignment: 1 + SectionData: 0000 symbols: - Name: "@comp.id" Value: 10394907 |