<div dir="ltr">You wrote different code for 32-bit than 64-bit, and that is why you needed it. I'm going to fix it and remove this test.</div><div class="gmail_extra"><br><div class="gmail_quote">On Wed, Apr 19, 2017 at 4:29 AM, George Rimar <span dir="ltr"><<a href="mailto:grimar@accesssoftek.com" target="_blank">grimar@accesssoftek.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">




<div dir="ltr" style="font-size:12pt;color:#000000;background-color:#ffffff;font-family:Calibri,Arial,Helvetica,sans-serif"><span class="">
<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>
</span><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_<wbr>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_<wbr>addralign);</div>
<div>  }​<span class="HOEnZb"><font color="#888888"><br>
</font></span></div><span class="HOEnZb"><font color="#888888">
<div><br>
</div>
</font></span></div><span class="HOEnZb"><font color="#888888">
<div class="gmail_extra">
<div class="gmail_quote">George.<br>
</div>
</div>
</font></span></div>
</div>
</div>
</div>

</blockquote></div><br></div>