[PATCH] D31943: [Support] - Implemented zlib::StreamCompression class.

David Blaikie via llvm-commits llvm-commits at lists.llvm.org
Thu Apr 13 08:58:51 PDT 2017


On Thu, Apr 13, 2017 at 1:32 AM George Rimar <grimar at accesssoftek.com>
wrote:

> >Fair, for sure and I /think/ George agrees - this review is totally
> pending until a discussion has demonstrated a need for this >code.
>
> I think you mentioned that implemented functionality could be useful for llvm-dwp
> ?
> Not sure what exactly functionality you may need there, but if compression
> without decompression, like
> this patch do may be useful, then why not to continue it ?
>

Because I don't have any plans to actually work on that in the
near/immediate future - so this would be dead code.

I wouldn't submit this for that use case unless someone has an immediate
use for it - which I don't.

If/when I do need it, if it hasn't been added for other needs like lld, I
can resurrect this review thread, etc.


>
> George.
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20170413/5a2f02b4/attachment.html>


More information about the llvm-commits mailing list