[PATCH] D126793: [ELF] Remove support for legacy .zdebug sections

[Peter Smith via Phabricator via llvm-commits]

peter.smith accepted this revision.
peter.smith added a comment.
This revision is now accepted and ready to land.

I'm happy with removing given that there is an official ELF replacement. Will be worth waiting a bit of time before committing to see if there are any last minute objections.



================
Comment at: lld/ELF/InputSection.cpp:73
 
-  // In ELF, each section can be compressed by zlib, and if compressed,
-  // section name may be mangled by appending "z" (e.g. ".zdebug_info").
-  // If that's the case, demangle section name so that we can handle a
-  // section as if it weren't compressed.
-  if ((flags & SHF_COMPRESSED) || name.startswith(".zdebug")) {
+  // If SHF_COMPRESSED is set, parse the header. The legacy .zdebug format is no
+  // longer supported.
----------------
Possibly worth a warning message if a section prefix of `.zebug` is encountered? For example:
```LLD does not support legacy .zdebug compressed debug sections produced with -gz=zlib-gnu. Please rebuild with -gz=zlib.```
  


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D126793