[PATCH] D27496: [ELF] Only binaries should have DT_DEBUG entry

[Petr Hosek via Phabricator via llvm-commits]

phosek added inline comments.


================
Comment at: ELF/SyntheticSections.cpp:798
 
-  if (!Config->Entry.empty())
+  if (!Config->Shared && !Config->Relocatable)
     add({DT_DEBUG, (uint64_t)0});
----------------
ruiu wrote:
> Do you actually want to exclude DSOs?
Yes, only leafs (that is statically or dynamically linked executables) should have `DT_DEBUG` entry so this should be correct.


Repository:
  rL LLVM

https://reviews.llvm.org/D27496