From 7ebff6ab26aa03423c61e0370377f11725184199 Mon Sep 17 00:00:00 2001 From: Alvin Wong Date: Wed, 28 Sep 2022 12:45:23 +0300 Subject: [PATCH] [lldb][COFF] Load absolute symbols from COFF symbol table Reviewed By: labath Differential Revision: https://reviews.llvm.org/D134517 --- .../Plugins/ObjectFile/PECOFF/ObjectFilePECOFF.cpp | 3 +++ lldb/test/Shell/ObjectFile/PECOFF/symbol.yaml | 13 ++++++++++--- 2 files changed, 13 insertions(+), 3 deletions(-) diff --git a/lldb/source/Plugins/ObjectFile/PECOFF/ObjectFilePECOFF.cpp b/lldb/source/Plugins/ObjectFile/PECOFF/ObjectFilePECOFF.cpp index 0a2777a193c8..ff824332bdd8 100644 --- a/lldb/source/Plugins/ObjectFile/PECOFF/ObjectFilePECOFF.cpp +++ b/lldb/source/Plugins/ObjectFile/PECOFF/ObjectFilePECOFF.cpp @@ -830,6 +830,9 @@ void ObjectFilePECOFF::AppendFromCOFFSymbolTable( symbol.SetType(exported->GetType()); } } + } else if (section_number == llvm::COFF::IMAGE_SYM_ABSOLUTE) { + symbol.GetAddressRef() = Address(coff_sym_ref.getValue()); + symbol.SetType(lldb::eSymbolTypeAbsolute); } symtab.AddSymbol(symbol); } diff --git a/lldb/test/Shell/ObjectFile/PECOFF/symbol.yaml b/lldb/test/Shell/ObjectFile/PECOFF/symbol.yaml index 8bda82c6b4a0..4a663a84a493 100644 --- a/lldb/test/Shell/ObjectFile/PECOFF/symbol.yaml +++ b/lldb/test/Shell/ObjectFile/PECOFF/symbol.yaml @@ -4,9 +4,10 @@ ## The .file symbol isn't checked, but is included to test that the symbol ## table iteration handles cases with a symbol with more than one aux symbol. -# CHECK: Type File Address/Value {{.*}} Size Flags Name -# CHECK: Code 0x0000000040001000 0x{{[0-9a-f]+}} 0x{{[0-9a-f]+}} entry -# CHECK: 0x0000000040002000 0x{{[0-9a-f]+}} 0x{{[0-9a-f]+}} variable +# CHECK: Type File Address/Value {{.*}} Size Flags Name +# CHECK: Code 0x0000000040001000 0x{{[0-9a-f]+}} 0x{{[0-9a-f]+}} entry +# CHECK: 0x0000000040002000 0x{{[0-9a-f]+}} 0x{{[0-9a-f]+}} variable +# CHECK: Absolute 0x00000000deadbeef 0x{{[0-9a-f]+}} 0x{{[0-9a-f]+}} absolute_symbol --- !COFF OptionalHeader: @@ -123,4 +124,10 @@ symbols: SimpleType: IMAGE_SYM_TYPE_NULL ComplexType: IMAGE_SYM_DTYPE_NULL StorageClass: IMAGE_SYM_CLASS_EXTERNAL + - Name: absolute_symbol + Value: 0xdeadbeef + SectionNumber: -1 + SimpleType: IMAGE_SYM_TYPE_NULL + ComplexType: IMAGE_SYM_DTYPE_NULL + StorageClass: IMAGE_SYM_CLASS_STATIC ...