[lld] r300445 - [ELF] - Add compress-debug-output-i386.s testcase.

[George Rimar via llvm-commits]

?> Please revert this file. The -compress-debug-section flag is not an architecture-dependent feature. One test is enough.

But compressed section header is different for 32/64 bit. In current code without the test, else branch is uncovered:

  if (Config->Is64) {
    Buf += sizeof(Elf64_Word); // Skip ch_reserved field.
    write64<E>(Buf, Size);
    Buf += sizeof(ELFT::Chdr::ch_size);
    write64<E>(Buf, Alignment);
    Buf += sizeof(ELFT::Chdr::ch_addralign);
  } else {
    write32<E>(Buf, Size);
    Buf += sizeof(ELFT::Chdr::ch_size);
    write32<E>(Buf, Alignment);
    Buf += sizeof(ELFT::Chdr::ch_addralign);
  }?

George.
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20170419/6c2e96e8/attachment.html>