[Bug c++/35909] New: ice for legal code

[Bug c++/35909] New: ice for legal code

[dcb314 at hotmail dot com]

I just tried to compile the Suse Linux package tse3-0.3.1
with the GNU C compiler version 4.4 snapshot 20080404

The compiler said

 g++ -DHAVE_CONFIG_H -I. -I../../.. -I../../../src -O2 -g -fmessage-length=0
-D_FORTIFY_SOURCE=2 -W -Wall -ansi -pedantic -MT Alsa-0.9.lo -MD -MP -MF
.deps/Alsa-0.9.Tpo -c Alsa-0.9.cpp  -fPIC -DPIC -o .libs/Alsa-0.9.o
In file included from Alsa-0.9.cpp:35:
/usr/include/sys/asoundlib.h:1:2: warning: #warning This header is deprecated,
use <alsa/asoundlib.h> instead.
Alsa-0.9.cpp: In member function 'void
TSE3::Plt::AlsaMidiScheduler::getSystemInfo()':
Alsa-0.9.cpp:208: warning: the address of 'ci' will always evaluate as 'true'
Alsa-0.9.cpp:219: warning: the address of 'pi' will always evaluate as 'true'
Alsa-0.9.cpp:238: warning: the address of 'subs' will always evaluate as 'true'
Alsa-0.9.cpp: In member function 'void
TSE3::Plt::AlsaImpl::tx(TSE3::MidiCommand, int, long int, long int)':
Alsa-0.9.cpp:343: internal compiler error: in build_target_expr, at
cp/tree.c:267
Please submit a full bug report,
with preprocessed source if appropriate.
See <http://gcc.gnu.org/bugs.html> for instructions.

Preprocessed C++ source code attached. No flags required


-- 
           Summary: ice for legal code
           Product: gcc
           Version: 4.4.0
            Status: UNCONFIRMED
          Severity: normal
          Priority: P3
         Component: c++
        AssignedTo: unassigned at gcc dot gnu dot org
        ReportedBy: dcb314 at hotmail dot com
  GCC host triplet: suse-linux-x86_64


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

[Bug c++/35909] ice for legal code

[dcb314 at hotmail dot com]

------- Comment #1 from dcb314 at hotmail dot com  2008-04-11 18:08 -------
Created an attachment (id=15468)
 --> (http://gcc.gnu.org/bugzilla/attachment.cgi?id=15468&action=view)
C++ source code


-- 


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

[Bug c++/35909] [4.4 Regression] ice for legal code

[rguenth at gcc dot gnu dot org]

-- 

rguenth at gcc dot gnu dot org changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
           Keywords|                            |ice-on-valid-code
            Summary|ice for legal code          |[4.4 Regression] ice for
                   |                            |legal code
   Target Milestone|---                         |4.4.0


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

[Bug c++/35909] [4.3/4.4 Regression] ice for legal code

[pinskia at gcc dot gnu dot org]

------- Comment #2 from pinskia at gcc dot gnu dot org  2008-04-11 21:29 -------
This is most likely caused by PR 35708.


-- 

pinskia at gcc dot gnu dot org changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
                 CC|                            |jason at gcc dot gnu dot org
      Known to fail|                            |4.4.0
      Known to work|                            |4.3.0
            Summary|[4.4 Regression] ice for    |[4.3/4.4 Regression] ice for
                   |legal code                  |legal code
   Target Milestone|4.4.0                       |4.3.1


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

[Bug c++/35909] [4.4 Regression] ICE with bit-field and const references

[pinskia at gcc dot gnu dot org]

------- Comment #3 from pinskia at gcc dot gnu dot org  2008-04-12 22:03 -------
Reduced testcase:
struct MidiCommand
{
  unsigned data1 : 8;
};
void g(const unsigned char &);
void f(MidiCommand mc)
{
  g(mc.data1);
}

----- CUT ---
And I was wrong, it is not related to that PR.


-- 

pinskia at gcc dot gnu dot org changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
             Status|UNCONFIRMED                 |NEW
     Ever Confirmed|0                           |1
      Known to work|4.3.0                       |
   Last reconfirmed|0000-00-00 00:00:00         |2008-04-12 22:03:37
               date|                            |
            Summary|[4.3/4.4 Regression] ice for|[4.4 Regression] ICE with
                   |legal code                  |bit-field and const
                   |                            |references
   Target Milestone|4.3.1                       |4.4.0


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

[Bug c++/35909] [4.4 Regression] ICE with bit-field and const references

[pinskia at gcc dot gnu dot org]

------- Comment #4 from pinskia at gcc dot gnu dot org  2008-04-15 08:57 -------
It did work with:
GNU C++ (GCC) version 4.3.0 20080128 (experimental) [trunk revision 131899]
(i386-apple-darwin8.10.1)
But fails with:
GNU C++ (GCC) version 4.4.0 20080304 (experimental) [trunk revision 132852]
(i386-apple-darwin8.10.1)


-- 


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

[Bug c++/35909] [4.3/4.4 Regression] ICE with bit-field and const references

[rguenth at gcc dot gnu dot org]

------- Comment #5 from rguenth at gcc dot gnu dot org  2008-04-20 20:48 -------
This also fails on the 4.3 branch now which makes it a regression against
4.3.0.


-- 

rguenth at gcc dot gnu dot org changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
                 CC|                            |rguenth at gcc dot gnu dot
                   |                            |org
           Severity|normal                      |blocker
      Known to fail|4.4.0                       |4.4.0 4.3.1
      Known to work|                            |4.3.0
           Priority|P3                          |P1
            Summary|[4.4 Regression] ICE with   |[4.3/4.4 Regression] ICE
                   |bit-field and const         |with bit-field and const
                   |references                  |references
   Target Milestone|4.4.0                       |4.3.1


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

[Bug c++/35909] [4.3/4.4 Regression] ICE with bit-field and const references

[jakub at gcc dot gnu dot org]

------- Comment #6 from jakub at gcc dot gnu dot org  2008-04-21 15:58 -------
Introduced by PR35056, so it is not a regression against 4.3.0, but against
4.2.x.  The ICE is only present in ENABLE_CHECKING builds, otherwise
TARGET_EXPR
with incompatible type is created.
Why do you think the testcase is valid?


-- 

jakub at gcc dot gnu dot org changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
                 CC|                            |aoliva at gcc dot gnu dot
                   |                            |org


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

[Bug c++/35909] [4.3/4.4 Regression] ICE with bit-field and const references

[rguenth at gcc dot gnu dot org]

------- Comment #7 from rguenth at gcc dot gnu dot org  2008-04-21 16:04 -------
That explains why I didn't see it with my 4.3.0 build.  Lowering severity
again.


-- 

rguenth at gcc dot gnu dot org changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
           Severity|blocker                     |normal
           Keywords|                            |ice-checking
           Priority|P1                          |P2


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

[Bug c++/35909] [4.3/4.4 Regression] ICE with bit-field and const references

[aoliva at gcc dot gnu dot org]

------- Comment #8 from aoliva at gcc dot gnu dot org  2008-04-22 06:27 -------
Mine


-- 

aoliva at gcc dot gnu dot org changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
         AssignedTo|unassigned at gcc dot gnu   |aoliva at gcc dot gnu dot
                   |dot org                     |org
             Status|NEW                         |ASSIGNED


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

[Bug c++/35909] [4.3/4.4 Regression] ICE with bit-field and const references

[aoliva at gcc dot gnu dot org]

------- Comment #9 from aoliva at gcc dot gnu dot org  2008-04-22 06:28 -------
Created an attachment (id=15507)
 --> (http://gcc.gnu.org/bugzilla/attachment.cgi?id=15507&action=view)
Patch that fixes the bug, under test

Here's a patch that I've come up with that fixes this bug.  I've just started
testing it.


-- 


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

[Bug c++/35909] [4.3/4.4 Regression] ICE with bit-field and const references

[aoliva at gcc dot gnu dot org]

------- Comment #10 from aoliva at gcc dot gnu dot org  2008-05-27 05:48 -------
Subject: Bug 35909

Author: aoliva
Date: Tue May 27 05:48:02 2008
New Revision: 135969

URL: http://gcc.gnu.org/viewcvs?root=gcc&view=rev&rev=135969
Log:
gcc/cp/ChangeLog:
PR c++/35909
* call.c (convert_like_real): Convert bitfield to desired type
before creating temporary.
gcc/testsuite/ChangeLog:
PR c++/35909
* g++.dg/conversion/bitfield9.C: New.

Added:
    trunk/gcc/testsuite/g++.dg/conversion/bitfield9.C
Modified:
    trunk/gcc/cp/ChangeLog
    trunk/gcc/cp/call.c
    trunk/gcc/testsuite/ChangeLog


-- 


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

[Bug c++/35909] [4.3/4.4 Regression] ICE with bit-field and const references

[aoliva at gcc dot gnu dot org]

------- Comment #11 from aoliva at gcc dot gnu dot org  2008-05-27 06:03 -------
Subject: Bug 35909

Author: aoliva
Date: Tue May 27 06:02:18 2008
New Revision: 135970

URL: http://gcc.gnu.org/viewcvs?root=gcc&view=rev&rev=135970
Log:
gcc/cp/ChangeLog:
PR c++/35909
* call.c (convert_like_real): Convert bitfield to desired type
before creating temporary.
gcc/testsuite/ChangeLog:
PR c++/35909
* g++.dg/conversion/bitfield9.C: New.

Added:
    branches/gcc-4_3-branch/gcc/testsuite/g++.dg/conversion/bitfield9.C
      - copied unchanged from r135969,
trunk/gcc/testsuite/g++.dg/conversion/bitfield9.C
Modified:
    branches/gcc-4_3-branch/gcc/cp/ChangeLog
    branches/gcc-4_3-branch/gcc/cp/call.c
    branches/gcc-4_3-branch/gcc/testsuite/ChangeLog


-- 


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

[Bug c++/35909] [4.3/4.4 Regression] ICE with bit-field and const references

[aoliva at gcc dot gnu dot org]

------- Comment #12 from aoliva at gcc dot gnu dot org  2008-05-27 06:03 -------
Fixed.


-- 

aoliva at gcc dot gnu dot org changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
             Status|ASSIGNED                    |RESOLVED
         Resolution|                            |FIXED


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