[PATCH] D128667: [WIP] Add Zstd ELF support

Cole Kissane via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Wed Jun 29 11:36:21 PDT 2022


ckissane added a comment.

In D128667#3612956 <https://reviews.llvm.org/D128667#3612956>, @MaskRay wrote:

> The lld/ change should be separate. And it needs tests. We should have several tests such as mixed zlib and zstd input, malformed zstd input, compressed level, etc. `OutputSections` should not have a new member. The parallel output part will be tricky, too (see D117853 <https://reviews.llvm.org/D117853>). I can handle the lld/ELF part and leave other stuff to you.
>
> llvm-objcopy should probably be a separate patch as well.
>
> - clang (Driver, cc1as) and MC (including `llvm-mc --compress-debug-sections`) in one patch
> - lld/ELF
> - llvm-objcopy
>
>> Context not available.
>
> See https://llvm.org/docs/Phabricator.html#requesting-a-review-via-the-web-interface "To make reviews easier, please always include as much context as possible with your diff!"

@MaskRay I would gladly accept your help on lld/ELF and I'll break this patch up into 2 patches:

- clang (Driver, cc1as) and MC (including `llvm-mc --compress-debug-sections`) in one patch
- llvm-objcopy

as suggested.
Also, I believe you requested a git repo for the whole picture on adding zstd: https://github.com/ckissane/llvm-project/tree/ckissane.add-zstd-compression


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

https://reviews.llvm.org/D128667



More information about the cfe-commits mailing list