[PATCH] D55407: [COFF]Correct .eh_frame name, let llvm tools such as llvm-dwarfdump can dump .eh_frame info

Peiyuan Song via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Fri Dec 7 00:48:37 PST 2018


SquallATF updated this revision to Diff 177138.
SquallATF added a comment.

a better way map .eh_frame name


Repository:
  rL LLVM

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D55407/new/

https://reviews.llvm.org/D55407

Files:
  include/llvm/Object/COFF.h
  lib/Object/COFFObjectFile.cpp


Index: lib/Object/COFFObjectFile.cpp
===================================================================
--- lib/Object/COFFObjectFile.cpp
+++ lib/Object/COFFObjectFile.cpp
@@ -1284,6 +1284,12 @@
   return !DataDirectory;
 }
 
+StringRef COFFObjectFile::mapDebugSectionName(StringRef Name) const {
+  return StringSwitch<StringRef>(Name)
+      .Case("eh_fram", "eh_frame")
+      .Default(Name);
+}
+
 bool ImportDirectoryEntryRef::
 operator==(const ImportDirectoryEntryRef &Other) const {
   return ImportTable == Other.ImportTable && Index == Other.Index;
Index: include/llvm/Object/COFF.h
===================================================================
--- include/llvm/Object/COFF.h
+++ include/llvm/Object/COFF.h
@@ -1062,6 +1062,8 @@
   bool isRelocatableObject() const override;
   bool is64() const { return PE32PlusHeader; }
 
+  StringRef mapDebugSectionName(StringRef Name) const override;
+
   static bool classof(const Binary *v) { return v->isCOFF(); }
 };
 


-------------- next part --------------
A non-text attachment was scrubbed...
Name: D55407.177138.patch
Type: text/x-patch
Size: 979 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20181207/330961d8/attachment.bin>


More information about the llvm-commits mailing list