public inbox for gcc-bugs@sourceware.org help / color / mirror / Atom feed
From: "js at linuxtv dot org" <gcc-bugzilla@gcc.gnu.org> To: gcc-bugs@gcc.gnu.org Subject: [Bug c/30475] assert(int+100 > int) optimized away Date: Thu, 08 Mar 2007 01:03:00 -0000 [thread overview] Message-ID: <20070308010325.19181.qmail@sourceware.org> (raw) In-Reply-To: <bug-30475-3511@http.gcc.gnu.org/bugzilla/> ------- Comment #53 from js at linuxtv dot org 2007-03-08 01:03 ------- I read all this and the mailing list thread with great interest, however I think there is a fundamental flaw in the reasoning: C90 6.2.1.2 / C99 6.3.1.3 defines signed integer overflow as "implementation-defined behaviour", which is something completely different than "undefined behaviour". See C90 3.11 vs. 3.18 / C99 3.4.1 vs. 3.4.3. (3.4.1 implementation-defined behavior: "unspecified behavior where each implementation documents how the choice is made"). And lo and behold: http://gcc.gnu.org/onlinedocs/gcc-4.1.2/gcc/Integers-implementation.html http://gcc.gnu.org/onlinedocs/gcc-4.0.4/gcc/Integers-implementation.html "For conversion to a type of width N, the value is reduced modulo 2^N to be within range of the type; no signal is raised." (Older versions of gcc left this undocumented.) Of course, software which relies on that is non-portable, but that's different from being "invalid C". If it turns out that many major C compilers use a similar implementation (which is what I expect but have no proof of), then it would be wise if gcc would do the same. E.g. Sun C uses this definitions: http://docs.sun.com/source/819-3688/c90.implementation.app.html#pgfId-998756 "When an integer is converted to a shorter signed integer, the low order bits are copied from the longer integer to the shorter signed integer. The result may be negative." Anyway, gcc should behave as documented, which isn't the case at least for gcc (GCC) 4.1.2 20061115 (prerelease) (Debian 4.1.1-21). -- http://gcc.gnu.org/bugzilla/show_bug.cgi?id=30475
next prev parent reply other threads:[~2007-03-08 1:03 UTC|newest] Thread overview: 62+ messages / expand[flat|nested] mbox.gz Atom feed top 2007-01-15 19:43 [Bug c/30475] New: " felix-gcc at fefe dot de 2007-01-15 19:46 ` [Bug c/30475] " felix-gcc at fefe dot de 2007-01-15 19:47 ` pinskia at gcc dot gnu dot org 2007-01-15 19:50 ` felix-gcc at fefe dot de 2007-01-15 19:57 ` felix-gcc at fefe dot de 2007-01-15 20:04 ` pinskia at gcc dot gnu dot org 2007-01-16 4:47 ` pinskia at gcc dot gnu dot org 2007-01-16 7:00 ` pluto at agmk dot net 2007-01-16 7:24 ` gcc at mailinator dot com 2007-01-17 13:55 ` felix-gcc at fefe dot de 2007-01-17 14:26 ` rguenth at gcc dot gnu dot org 2007-01-17 14:31 ` rguenth at gcc dot gnu dot org 2007-01-17 15:21 ` felix-gcc at fefe dot de 2007-01-17 16:32 ` rguenth at gcc dot gnu dot org 2007-01-17 16:37 ` felix-gcc at fefe dot de 2007-01-17 16:54 ` erdgeist-gcc at erdgeist dot org 2007-01-17 16:57 ` pinskia at gcc dot gnu dot org 2007-01-17 17:03 ` felix-gcc at fefe dot de 2007-01-17 17:06 ` rguenth at gcc dot gnu dot org 2007-01-17 17:12 ` pinskia at gcc dot gnu dot org 2007-01-17 17:14 ` amacleod at redhat dot com 2007-01-17 17:20 ` felix-gcc at fefe dot de 2007-01-17 17:42 ` pinskia at gcc dot gnu dot org 2007-01-17 18:23 ` felix-gcc at fefe dot de 2007-01-17 18:43 ` pinskia at gcc dot gnu dot org 2007-01-17 19:04 ` felix-gcc at fefe dot de 2007-01-17 19:18 ` pinskia at gcc dot gnu dot org 2007-01-18 15:20 ` felix-gcc at fefe dot de 2007-01-18 15:24 ` felix-gcc at fefe dot de 2007-01-18 17:36 ` pinskia at gcc dot gnu dot org 2007-01-21 8:58 ` pinskia at gcc dot gnu dot org 2007-01-21 12:23 ` andreas at andreas dot org 2007-01-21 12:49 ` andreas at andreas dot org 2007-01-21 13:53 ` felix-gcc at fefe dot de 2007-01-21 16:31 ` pinskia at gcc dot gnu dot org 2007-01-21 17:30 ` andreas at andreas dot org 2007-01-21 17:47 ` felix-gcc at fefe dot de 2007-01-21 18:16 ` pluto at agmk dot net 2007-01-21 19:46 ` rguenth at gcc dot gnu dot org 2007-01-21 20:14 ` pinskia at gcc dot gnu dot org 2007-01-21 21:52 ` tromey at gcc dot gnu dot org 2007-01-22 2:18 ` felix-gcc at fefe dot de 2007-01-22 2:27 ` pinskia at gcc dot gnu dot org 2007-01-22 13:02 ` felix-gcc at fefe dot de 2007-01-22 17:15 ` pinskia at gcc dot gnu dot org 2007-01-22 18:26 ` steven at gcc dot gnu dot org 2007-01-22 18:33 ` pinskia at gcc dot gnu dot org 2007-01-22 18:36 ` andreas at andreas dot org 2007-01-22 19:50 ` felix-gcc at fefe dot de 2007-01-22 20:16 ` ian at airs dot com 2007-01-22 22:27 ` pinskia at gcc dot gnu dot org 2007-01-22 23:10 ` andreas at andreas dot org 2007-01-23 0:46 ` kargl at gcc dot gnu dot org 2007-03-08 1:03 ` js at linuxtv dot org [this message] 2007-03-08 1:14 ` pinskia at gcc dot gnu dot org 2007-03-08 16:23 ` js at linuxtv dot org [not found] <bug-30475-4@http.gcc.gnu.org/bugzilla/> 2014-02-16 10:00 ` jackie.rosen at hushmail dot com 2021-01-05 12:26 ` daniel.marjamaki at gmail dot com 2021-01-05 12:56 ` daniel.marjamaki at gmail dot com 2021-01-05 13:30 ` jakub at gcc dot gnu.org 2021-01-05 13:37 ` jakub at gcc dot gnu.org 2021-01-06 10:37 ` daniel.marjamaki at gmail dot com
Reply instructions: You may reply publicly to this message via plain-text email using any one of the following methods: * Save the following mbox file, import it into your mail client, and reply-to-all from there: mbox Avoid top-posting and favor interleaved quoting: https://en.wikipedia.org/wiki/Posting_style#Interleaved_style * Reply using the --to, --cc, and --in-reply-to switches of git-send-email(1): git send-email \ --in-reply-to=20070308010325.19181.qmail@sourceware.org \ --to=gcc-bugzilla@gcc.gnu.org \ --cc=gcc-bugs@gcc.gnu.org \ /path/to/YOUR_REPLY https://kernel.org/pub/software/scm/git/docs/git-send-email.html * If your mail client supports setting the In-Reply-To header via mailto: links, try the mailto: linkBe sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions for how to clone and mirror all data and code used for this inbox; as well as URLs for read-only IMAP folder(s) and NNTP newsgroup(s).