<div dir="ltr">I think the most important question is whether we want to support emitting compressed debug sections or not. Looks like reading compressed debug sections is at least more common.</div><div class="gmail_extra"><br><div class="gmail_quote">On Fri, May 13, 2016 at 6:26 AM, Rafael Espíndola <span dir="ltr"><<a href="mailto:rafael.espindola@gmail.com" target="_blank">rafael.espindola@gmail.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><span class="">> ruiu wrote:<br>
>> I think we don't have to care about this edge case. Unless you are compressing random bits, it usually shrink.<br>
> I would leave it as is. That is mentioned in docs: "The .debug_line and .debug_pubnames sections would be larger compressed than in their original uncompressed form, and have therefore been left uncompressed." (<a href="https://docs.oracle.com/cd/E53394_01/html/E54813/section_compression.html#scrolltoc" rel="noreferrer" target="_blank">https://docs.oracle.com/cd/E53394_01/html/E54813/section_compression.html#scrolltoc</a>) and done both by gold/bfd and very transparent in implementation.<br>
> In addition I think that if feature is used then motivation and expected result is to consume less space, I do not think it is acceptable to consume more than without it, it is confusing and opposite to its description.<br>
<br>
<br>
</span>I think we can start without this and see how many case we hit in the<br>
field when it gets bigger.<br>
<span class=""><br>
> ruiu wrote:<br>
>> Why we have two types of headers? If one is obsolete, we probably don't want to implement it.<br>
> There are 2 different approaches used (<a href="https://docs.oracle.com/cd/E53394_01/html/E54813/section_compression.html" rel="noreferrer" target="_blank">https://docs.oracle.com/cd/E53394_01/html/E54813/section_compression.html</a>):<br>
> 1) gnu-style renames sections to ".z*[NAME]" and do not uses SHF_COMPRESSED.<br>
> 2) zlib style. Sections have SHF_COMPRESSED flag and are not renamed.<br>
><br>
> "Unless there is a specific requirement to use the zlib-gnu style, the more general default zlib style is recommended." says <a href="https://docs.oracle.com/cd/E53394_01/html/E54813/man-cds.html" rel="noreferrer" target="_blank">https://docs.oracle.com/cd/E53394_01/html/E54813/man-cds.html</a>, but it is not mentioned if that depricated or something.<br>
> Not supporting of zlib-gnu style would make implementation a bit cleaner.<br>
><br>
> It is not hard to support both though.<br>
<br>
</span>Lets please start without support of the gnu style.<br>
<br>
Cheers,<br>
Rafael<br>
</blockquote></div><br></div>