<html>
<head>
<meta http-equiv="Content-Type" content="text/html; charset=koi8-r">
<style type="text/css" style="display:none"><!-- p { margin-top: 0px; margin-bottom: 0px; }--></style>
</head>
<body dir="ltr" style="font-size:12pt;color:#000000;background-color:#FFFFFF;font-family:Calibri,Arial,Helvetica,sans-serif;">
<p>> <span style="color: rgb(33, 33, 33); font-size: 12pt;">Please revert this file. The -compress-debug-section flag is not an architecture-dependent feature. One test is enough.</span><br>
</p>
<div style="color: rgb(33, 33, 33);">
<div>
<div dir="ltr">
<div class="gmail_extra"><br>
</div>
<div class="gmail_extra">But compressed section header is different for 32/64 bit. In current code without the test, else branch is uncovered:<br>
</div>
<div class="gmail_extra"><br>
</div>
<div class="gmail_extra">
<div> if (Config->Is64) {</div>
<div> Buf += sizeof(Elf64_Word); // Skip ch_reserved field.</div>
<div> write64<E>(Buf, Size);</div>
<div> Buf += sizeof(ELFT::Chdr::ch_size);</div>
<div> write64<E>(Buf, Alignment);</div>
<div> Buf += sizeof(ELFT::Chdr::ch_addralign);</div>
<div> } else {</div>
<div> write32<E>(Buf, Size);</div>
<div> Buf += sizeof(ELFT::Chdr::ch_size);</div>
<div> write32<E>(Buf, Alignment);</div>
<div> Buf += sizeof(ELFT::Chdr::ch_addralign);</div>
<div> }<br>
</div>
<div><br>
</div>
</div>
<div class="gmail_extra">
<div class="gmail_quote">George.<br>
</div>
</div>
</div>
</div>
</div>
</body>
</html>