[Bug binutils/21212] Improve "Empty loadable segment detected" warning

[binutils at ineranves dot de]

https://sourceware.org/bugzilla/show_bug.cgi?id=21212

--- Comment #3 from Martin Bickel <binutils at ineranves dot de> ---
Hi Nick,

your updated change is ok - I don't see any value in having segments with
memsz==0 , so a warning makes sense.

However, your commit message and Changelog entry is the wrong way round: the
warning is disabled when filesz is zero and *memsz* is nonzero.



Here is the behavior:

filesz | memsz | original code | Martin's patch | Nick's patch
--------------------------------------------------------------
  0       0      warn             don't warn       warn
  0     > 0      warn             don't warn       don't warn
> 0     > 0      warn             warn             warn
> 0       0      warn             warn             warn

The last line shouldn't be relevant as it's illegal in ELF.

-- 
You are receiving this mail because:
You are on the CC list for the bug.