[PATCH] D38078: [dsymutil] Don't resolve DIE reference to NULL DIE.
Jonas Devlieghere via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Thu Sep 21 03:30:16 PDT 2017
This revision was automatically updated to reflect the committed changes.
Closed by commit rL313872: [dsymutil] Don't resolve DIE reference to NULL DIE. (authored by JDevlieghere).
Changed prior to commit:
https://reviews.llvm.org/D38078?vs=116026&id=116162#toc
Repository:
rL LLVM
https://reviews.llvm.org/D38078
Files:
llvm/trunk/test/tools/dsymutil/Inputs/null_die.o
llvm/trunk/test/tools/dsymutil/null-die.test
llvm/trunk/tools/dsymutil/DwarfLinker.cpp
Index: llvm/trunk/test/tools/dsymutil/null-die.test
===================================================================
--- llvm/trunk/test/tools/dsymutil/null-die.test
+++ llvm/trunk/test/tools/dsymutil/null-die.test
@@ -0,0 +1,41 @@
+#RUN: llvm-dsymutil -f -oso-prepend-path=%p/Inputs/ -y %s -no-output 2>&1 \
+#RUN: | FileCheck %s
+
+# CHECK: warning: could not find referenced DIE
+
+# We've modified the DW_AT_abstract_origin offset to reference a NULL DIE.
+#
+# Source:
+#
+# void f1() {}
+# __attribute__((always_inline)) void f2() {
+# f1();
+# }
+# int main() {
+# f2();
+# }
+#
+# Compile with:
+#
+# $ clang -g null_die.c -O0 -S -o null_die.s
+#
+# Manually patch the DW_AT_abstract_origin to point to a NULL DIE.
+#
+# $ llvm-mc -triple x86_64-apple-darwin -filetype=obj -o null_die.o null_die.s
+#
+# To generate the debug map:
+#
+# $ ld -arch x86_64 -macosx_version_min 10.13.0 -lSystem null_die.o -o null_die
+# $ llvm-dsymutil -dump-debug-map null_die
+
+---
+triple: 'x86_64-apple-darwin'
+binary-path: null_die
+objects:
+ - filename: /null_die.o
+ timestamp: 1505928022
+ symbols:
+ - { sym: _main, objAddr: 0x0000000000000020, binAddr: 0x0000000100000F10, size: 0x0000000D }
+ - { sym: _f1, objAddr: 0x0000000000000000, binAddr: 0x0000000100000EF0, size: 0x00000010 }
+ - { sym: _f2, objAddr: 0x0000000000000010, binAddr: 0x0000000100000F00, size: 0x00000010 }
+...
Index: llvm/trunk/tools/dsymutil/DwarfLinker.cpp
===================================================================
--- llvm/trunk/tools/dsymutil/DwarfLinker.cpp
+++ llvm/trunk/tools/dsymutil/DwarfLinker.cpp
@@ -1495,8 +1495,12 @@
uint64_t RefOffset = *RefValue.getAsReference();
if ((RefCU = getUnitForOffset(Units, RefOffset)))
- if (const auto RefDie = RefCU->getOrigUnit().getDIEForOffset(RefOffset))
- return RefDie;
+ if (const auto RefDie = RefCU->getOrigUnit().getDIEForOffset(RefOffset)) {
+ // In a file with broken references, an attribute might point to a NULL
+ // DIE.
+ if(!RefDie.isNULL())
+ return RefDie;
+ }
Linker.reportWarning("could not find referenced DIE", &DIE);
return DWARFDie();
-------------- next part --------------
A non-text attachment was scrubbed...
Name: D38078.116162.patch
Type: text/x-patch
Size: 2227 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20170921/53b8aa89/attachment.bin>
More information about the llvm-commits
mailing list