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. Got it, I have also removed the extra member. In D128667#3612956 <https://reviews.llvm.org/D128667#3612956>, @MaskRay wrote: > 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. Sounds good, and thank you for your offer to help! In D128667#3612956 <https://reviews.llvm.org/D128667#3612956>, @MaskRay wrote: > llvm-objcopy should probably be a separate patch as well. got it. > - 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!" Doing so now CHANGES SINCE LAST ACTION https://reviews.llvm.org/D128667/new/ https://reviews.llvm.org/D128667 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
