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 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
