[PATCH] D61689: Change -gz and -Wa,--compress-debug-sections to use gABI compression (SHF_COMPRESSED)

Jordan Rupprecht via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Tue May 14 12:41:55 PDT 2019


rupprecht added a comment.

So, while I think this is an //entirely// reasonable assumption in most cases, it's also not one that provides any kind of workaround for the few cases where it's not universally true.

- As mentioned in the patch, this effectively changes the default from `-gz=zlib-gnu` to `-gz=zlib`. Anyone with an older toolchain that wants the old behavior can set `-gz=zlib-gnu`. Seems OK so far.
- However, passing the `-gz` flag //also// implies sending `--compress-debug-sections=XXX` to the assembler, which -- if someone is using `-no-integrated-as` has an older toolchain -- is not a supported option (at least the variant that takes a value).

In other works, for the few users that have an older toolchain, this provides literally no workaround. Not setting the flag causes failures with an older linker, and setting the flag causes failures with the assembler.

(echristo reverted this in rL360703 <https://reviews.llvm.org/rL360703>, I wanted to add some context)


Repository:
  rL LLVM

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

https://reviews.llvm.org/D61689





More information about the cfe-commits mailing list