From: Aidan Dodds Date: Thu, 28 May 2015 15:37:01 +0000 (+0000) Subject: Fix THUMB function detection when function name is not prefixed. X-Git-Tag: llvmorg-3.7.0-rc1~3598 X-Git-Url: http://review.tizen.org/git/?a=commitdiff_plain;h=5f2d0c3c23d1287522c20bb59337eb3ed9147511;p=platform%2Fupstream%2Fllvm.git Fix THUMB function detection when function name is not prefixed. Differential Revision: http://reviews.llvm.org/D10062 llvm-svn: 238433 --- diff --git a/lldb/source/Plugins/ObjectFile/ELF/ObjectFileELF.cpp b/lldb/source/Plugins/ObjectFile/ELF/ObjectFileELF.cpp index c043628..d8e35b8 100644 --- a/lldb/source/Plugins/ObjectFile/ELF/ObjectFileELF.cpp +++ b/lldb/source/Plugins/ObjectFile/ELF/ObjectFileELF.cpp @@ -1915,18 +1915,25 @@ ObjectFileELF::ParseSymbols (Symtab *symtab, if (arch.GetMachine() == llvm::Triple::arm) { - // THUMB functions have the lower bit of their address set. Fixup - // the actual address and mark the symbol as THUMB. - if (symbol_type == eSymbolTypeCode && symbol.st_value & 1) + if (symbol_type == eSymbolTypeCode) { - // Substracting 1 from the address effectively unsets - // the low order bit, which results in the address - // actually pointing to the beginning of the symbol. - // This delta will be used below in conjuction with - // symbol.st_value to produce the final symbol_value - // that we store in the symtab. - symbol_value_offset = -1; - additional_flags = ARM_ELF_SYM_IS_THUMB; + if (symbol.st_value & 1) + { + // Subtracting 1 from the address effectively unsets + // the low order bit, which results in the address + // actually pointing to the beginning of the symbol. + // This delta will be used below in conjunction with + // symbol.st_value to produce the final symbol_value + // that we store in the symtab. + symbol_value_offset = -1; + additional_flags = ARM_ELF_SYM_IS_THUMB; + m_address_class_map[symbol.st_value^1] = eAddressClassCodeAlternateISA; + } + else + { + // This address is ARM + m_address_class_map[symbol.st_value] = eAddressClassCode; + } } } }