[PATCH] D70658: [LLD][ELF] - Make compression level be dependent on -On.

[Fangrui Song via Phabricator via llvm-commits]

MaskRay added a comment.

https://bugs.llvm.org/show_bug.cgi?id=44089

> So, I think we should use level 1 for the default and level 6 for -O2, and I don't think we need a new option to specify some other value as a compression level.



  -O0 => level 1
  -O1 => level 1
  -O2 => level 6

makes sense.

We use `-Wl,--compress-debug-sections=zlib` and default `-Wl,-O1` for asan builds, so some users may observe size increases (due to: zlib level 6 -> zlib level 1). I will try accommodating the targets not to rely on the size, but I may request for a new option for compression level (@ruiu) if fixing them is not feasible. (I genuinely want to keep the configuration simple, so not having an additional option is also my preference.)


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

https://reviews.llvm.org/D70658