public inbox for gcc-bugs@sourceware.org
help / color / mirror / Atom feed
From: "pinskia at gmail dot com" <gcc-bugzilla@gcc.gnu.org>
To: gcc-bugs@gcc.gnu.org
Subject: [Bug c++/37281] bad code generation with enum and -m32
Date: Fri, 29 Aug 2008 16:41:00 -0000	[thread overview]
Message-ID: <20080829163947.930.qmail@sourceware.org> (raw)
In-Reply-To: <bug-37281-9476@http.gcc.gnu.org/bugzilla/>



------- Comment #1 from pinskia at gmail dot com  2008-08-29 16:39 -------
Subject: Re:   New: bad code generation with enum and -m32

This code is undefined as the value of  32 is outside the range of the  
enum.

Sent from my iPhone

On Aug 29, 2008, at 9:29, "gmorin1 at bloomberg dot net"
<gcc-bugzilla@gcc.gnu.org 
 > wrote:

> This is a copy of a Debian report that I made (I thought I might as  
> well report
> it directly here too): http://bugs.debian.org/cgi-bin/bugreport.cgi?bug=496965
>
> g++ generates bad code for this program on an amd64 machine.  It  
> works with
> fine in 64-bit and with older versions of gcc.
>
> #include <stdlib.h>
> struct MyTypes {
>    enum Type {
>        NOVALUE = 1,
>        VALUE   = 30,
>        VALUE2 = 31,
>        VALUE3 = -3
>    };
>
>    static bool isValue(MyTypes::Type value);
> };
>
> bool MyTypes::isValue(MyTypes::Type value)
> {
>    switch (value) {
>      case MyTypes::VALUE:
>      case MyTypes::VALUE2:
>      case MyTypes::VALUE3: {
>        return true;
>      }
>      default: {
>        return false;
>      }
>    }
> }
>
> int main(int argc, char *argv[])
> {
>    if (true == MyTypes::isValue(MyTypes::Type(32))) {
>        abort();
>    }
>    return 0;
> }
>
> guillaum@canard:~$ g++ -m32 -Wall -o e enumtest.cpp
> guillaum@canard:~$ ./e
> Aborted
> guillaum@canard:~$ g++ -Wall -o e enumtest.cpp
> guillaum@canard:~$ ./e
> guillaum@canard:~$ g++-4.2 -m32 -Wall -o e enumtest.cpp
> guillaum@canard:~$ ./e
> guillaum@canard:~$ g++-4.1 -m32 -Wall -o e enumtest.cpp
> guillaum@canard:~$ ./e
>
> It works with -O2 though (not -O1):
> guillaum@canard:~$ g++ -O2 -m32 -Wall -o e enumtest.cpp
> guillaum@canard:~$ ./e
>
>
> -- System Information:
> Debian Release: lenny/sid
>  APT prefers testing
>  APT policy: (990, 'testing'), (500, 'unstable')
> Architecture: amd64 (x86_64)
>
>
> -- 
>           Summary: bad code generation with enum and -m32
>           Product: gcc
>           Version: 4.3.1
>            Status: UNCONFIRMED
>          Severity: normal
>          Priority: P3
>         Component: c++
>        AssignedTo: unassigned at gcc dot gnu dot org
>        ReportedBy: gmorin1 at bloomberg dot net
>
>
> http://gcc.gnu.org/bugzilla/show_bug.cgi?id=37281
>


-- 


http://gcc.gnu.org/bugzilla/show_bug.cgi?id=37281


  parent reply	other threads:[~2008-08-29 16:41 UTC|newest]

Thread overview: 6+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2008-08-29 16:30 [Bug c++/37281] New: " gmorin1 at bloomberg dot net
2008-08-29 16:39 ` Andrew Thomas Pinski
2008-08-29 16:41 ` pinskia at gmail dot com [this message]
2008-08-29 16:59 ` [Bug c++/37281] " gmorin1 at bloomberg dot net
2008-08-29 17:02 ` manu at gcc dot gnu dot org
2010-05-04  4:49 ` jason at gcc dot gnu dot org

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=20080829163947.930.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: link
Be 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).