[PATCH] D68133: [Symbolize] Use the local MSVC C++ demangler instead of relying on dbghelp. NFC.
Martin Storsjö via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Fri Sep 27 05:55:30 PDT 2019
mstorsjo created this revision.
mstorsjo added a reviewer: rnk.
Herald added subscribers: rupprecht, erik.pilkington, hiraditya.
Herald added a project: LLVM.
This allows making a couple llvm-symbolizer tests run in all environments.
Keeping the existing exact prefix checks to make sure this is a strict NFC patch.
Repository:
rL LLVM
https://reviews.llvm.org/D68133
Files:
llvm/lib/DebugInfo/Symbolize/Symbolize.cpp
llvm/test/tools/llvm-symbolizer/coff-dwarf.test
llvm/test/tools/llvm-symbolizer/coff-exports.test
Index: llvm/test/tools/llvm-symbolizer/coff-exports.test
===================================================================
--- llvm/test/tools/llvm-symbolizer/coff-exports.test
+++ llvm/test/tools/llvm-symbolizer/coff-exports.test
@@ -5,11 +5,6 @@
RUN: llvm-symbolizer 0x500A 0x5038 0x504B -i --relative-address -obj="%p/Inputs/coff-exports.exe" \
RUN: | FileCheck %s
-This test relies on UnDecorateSymbolName, which is Win32-only.
-REQUIRES: system-windows
-REQUIRES: target-windows
-FIXME: This test depends on host, not target.
-
We get the expected stack trace, except 'foo' appears for the 'bar' frame
because 'bar' isn't in the export table.
Index: llvm/test/tools/llvm-symbolizer/coff-dwarf.test
===================================================================
--- llvm/test/tools/llvm-symbolizer/coff-dwarf.test
+++ llvm/test/tools/llvm-symbolizer/coff-dwarf.test
@@ -5,9 +5,6 @@
RUN: llvm-symbolizer 0x5009 0x5038 -i --relative-address -obj="%p/Inputs/coff-dwarf.exe" \
RUN: | FileCheck %s
-This test relies on UnDecorateSymbolName, which is Windows-only.
-REQUIRES: target-windows, system-windows
-
CHECK: foo(void)
CHECK: coff-dwarf.cpp:7
CHECK: bar(void)
Index: llvm/lib/DebugInfo/Symbolize/Symbolize.cpp
===================================================================
--- llvm/lib/DebugInfo/Symbolize/Symbolize.cpp
+++ llvm/lib/DebugInfo/Symbolize/Symbolize.cpp
@@ -35,19 +35,6 @@
#include <cassert>
#include <cstring>
-#if defined(_MSC_VER)
-#include <Windows.h>
-
-// This must be included after windows.h.
-#include <DbgHelp.h>
-#pragma comment(lib, "dbghelp.lib")
-
-// Windows.h conflicts with our COFF header definitions.
-#ifdef IMAGE_FILE_MACHINE_I386
-#undef IMAGE_FILE_MACHINE_I386
-#endif
-#endif
-
namespace llvm {
namespace symbolize {
@@ -524,31 +511,11 @@
const SymbolizableModule *DbiModuleDescriptor) {
// We can spoil names of symbols with C linkage, so use an heuristic
// approach to check if the name should be demangled.
- if (Name.substr(0, 2) == "_Z") {
- int status = 0;
- char *DemangledName = itaniumDemangle(Name.c_str(), nullptr, nullptr, &status);
- if (status != 0)
- return Name;
- std::string Result = DemangledName;
- free(DemangledName);
- return Result;
- }
+ // MSVC C++ mangled symbols start with '?', while itanium mangled ones
+ // start with _Z.
+ if (Name.substr(0, 2) == "_Z" || (!Name.empty() && Name.front() == '?'))
+ return demangle(Name);
-#if defined(_MSC_VER)
- if (!Name.empty() && Name.front() == '?') {
- // Only do MSVC C++ demangling on symbols starting with '?'.
- char DemangledName[1024] = {0};
- DWORD result = ::UnDecorateSymbolName(
- Name.c_str(), DemangledName, 1023,
- UNDNAME_NO_ACCESS_SPECIFIERS | // Strip public, private, protected
- UNDNAME_NO_ALLOCATION_LANGUAGE | // Strip __thiscall, __stdcall, etc
- UNDNAME_NO_THROW_SIGNATURES | // Strip throw() specifications
- UNDNAME_NO_MEMBER_TYPE | // Strip virtual, static, etc specifiers
- UNDNAME_NO_MS_KEYWORDS | // Strip all MS extension keywords
- UNDNAME_NO_FUNCTION_RETURNS); // Strip function return types
- return (result == 0) ? Name : std::string(DemangledName);
- }
-#endif
if (DbiModuleDescriptor && DbiModuleDescriptor->isWin32Module())
return std::string(demanglePE32ExternCFunc(Name));
return Name;
-------------- next part --------------
A non-text attachment was scrubbed...
Name: D68133.222150.patch
Type: text/x-patch
Size: 3452 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20190927/bf8cbac9/attachment-0001.bin>
More information about the llvm-commits
mailing list