git/list[1] front-page[2] threads[3] people[4] search[5] about
 

[Bug?] compiler warning with gcc >= 4.2

From
Frank Lichtenheld <frank@lichtenheld.de>
Date
Jul 3, 2007, 12:58 UTC
Message-ID
<20070703125844.GL12721@planck.djpig.de>
Hi,

while testing for an (probably) unrelated miscompilation bug, I got the following warning while compiling git:

gcc-4.2 -o sha1_file.o -c -g -O2 -Wall -DSHA1_HEADER='<openssl/sha.h>' -DETC_GITCONFIG='"/home/djpig/etc/gitconfig"' -DNO_STRLCPY sha1_file.c sha1_file.c: In function ‘check_packed_git_idx’: sha1_file.c:523: warning: assuming signed overflow does not occur when assuming that (X + c) < X is always false sha1_file.c:523: warning: assuming signed overflow does not occur when assuming that (X + c) < X is always false

This comes from the new -Wstrict-overflow which warns about the fact that with -fstrict-overflow, which is activated by default with -O2, the if clause referenced in the warning will be optimised away.

So I would be interested to know
a) if the compiler optimising this check away (which seems to be a check
   about whether signed overflow can occour) can lead to unwanted results
b) if not a), if it would make sense trying to suppress that warning, so
   that other people don't end up wondering the same as me
The used compiler was:
$ gcc-4.2 -v
Using built-in specs.
Target: hppa-linux-gnu
Configured with: ../src/configure -v
--enable-languages=c,c++,fortran,objc,obj-c++,treelang --prefix=/usr
--enable-shared --with-system-zlib --libexecdir=/usr/lib
--without-included-gettext --enable-threads=posix --enable-nls
--with-gxx-include-dir=/usr/include/c++/4.2 --program-suffix=-4.2
--enable-clocale=gnu --enable-libstdcxx-debug --enable-mpfr
--disable-libssp --disable-werror --enable-checking=release
--build=hppa-linux-gnu --host=hppa-linux-gnu
--target=hppa-linux-gnuThread model: posix
gcc version 4.2.1 20070627 (prerelease) (Debian 4.2-20070627-1)
Gruesse,
-- 
Frank Lichtenheld <frank@lichtenheld.de>
www: http://www.djpig.de/
Next: Nicolas Pitre
Message 1 of 3 in “[Bug?] compiler warning with gcc >= 4.2”
  1. Frank LichtenheldJul 3, 2007
  2. Nicolas PitreJul 3, 2007
  3. Jan HudecJul 6, 2007

Read the whole thread, see it on lore, or plain text.

$ cat FOOTERMessages come from the public archive at lore.kernel.org/git, fetched every hour. The front page is chosen and written each morning by an AI editor and can be wrong; the threads themselves are the record. About and API. For agents: an MCP server at https://gitlist.dev/mcp, and any thread, story or person page as Markdown by adding .md to its URL (or sending Accept: text/markdown). Details in /llms.txt.