[Bug c++/38705] New: ICE: canonical types differ for identical types const int and const AlpsNodeIndex_t

[Bug c++/38705] New: ICE: canonical types differ for identical types const int and const AlpsNodeIndex_t

[sdirkse at gams dot com]

I turfed up an ICE with optimization, but the thing compiled fine without.  To
reproduce, use the soon-to-be attached source and do:
madison.gams.com:/home/steve/lang/cpp/alpsBug$g++ -c AlpsTreeNode.ii 
madison.gams.com:/home/steve/lang/cpp/alpsBug$g++ -O3 -c AlpsTreeNode.ii 
AlpsTreeNode.cpp: In member function 'AlpsReturnStatus
AlpsTreeNode::encodeAlps(AlpsEncoded*) const':
AlpsTreeNode.cpp:144: internal compiler error: canonical types differ for
identical types const int and const AlpsNodeIndex_t
Please submit a full bug report,
with preprocessed source if appropriate.
See <http://gcc.gnu.org/bugs.html> for instructions.

I was using the trunk src at rev 142984:
$gcc -v
Using built-in specs.
Target: x86_64-unknown-linux-gnu
Configured with: ../configure --prefix=/usr/local2
--enable-languages=c,c++,fortran
Thread model: posix
gcc version 4.4.0 20081231 (experimental) (GCC)


-- 
           Summary: ICE: canonical types differ for identical types const
                    int and const AlpsNodeIndex_t
           Product: gcc
           Version: 4.4.0
            Status: UNCONFIRMED
          Severity: normal
          Priority: P3
         Component: c++
        AssignedTo: unassigned at gcc dot gnu dot org
        ReportedBy: sdirkse at gams dot com
GCC target triplet: x86_64-unknown-linux-gnu


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

[Bug c++/38705] ICE: canonical types differ for identical types const int and const AlpsNodeIndex_t

[sdirkse at gams dot com]

------- Comment #1 from sdirkse at gams dot com  2009-01-02 16:49 -------
Created an attachment (id=17023)
 --> (http://gcc.gnu.org/bugzilla/attachment.cgi?id=17023&action=view)
test case


-- 


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

[Bug c++/38705] ICE: canonical types differ for identical types const int and const AlpsNodeIndex_t

[pinskia at gcc dot gnu dot org]

------- Comment #2 from pinskia at gcc dot gnu dot org  2009-01-02 17:29 -------
Reducing ...


-- 


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

[Bug c++/38705] [4.4 Regression] ICE: canonical types differ for identical types const int and const AlpsNodeIndex_t

[pinskia at gcc dot gnu dot org]

------- Comment #3 from pinskia at gcc dot gnu dot org  2009-01-02 18:38 -------
Reduced testcase:
typedef int AlpsNodeIndex_t;
  void writeRep(char* representation_, const int len)   {
    __builtin_memcpy(representation_ , &len, sizeof(int));
  }
struct AlpsTreeNode
{
  AlpsNodeIndex_t index_;
  int explicit_;
  int encodeAlps() const;
  template <class T> void writeRep(const T& value) const {      }
};
int AlpsTreeNode:: encodeAlps()     const
{
  writeRep(explicit_);
  writeRep(index_);
}

--- CUT ---
I think this might have something to do with the __builtin_memcpy optimization.


-- 

pinskia at gcc dot gnu dot org changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
                 CC|                            |jakub at gcc dot gnu dot org
             Status|UNCONFIRMED                 |NEW
     Ever Confirmed|0                           |1
 GCC target triplet|x86_64-unknown-linux-gnu    |
           Keywords|                            |ice-checking, ice-on-valid-
                   |                            |code
   Last reconfirmed|0000-00-00 00:00:00         |2009-01-02 18:38:20
               date|                            |
            Summary|ICE: canonical types differ |[4.4 Regression] ICE:
                   |for identical types const   |canonical types differ for
                   |int and const               |identical types const int
                   |AlpsNodeIndex_t             |and const AlpsNodeIndex_t
   Target Milestone|---                         |4.4.0


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

[Bug c++/38705] [4.4 Regression] ICE: canonical types differ for identical types const int and const AlpsNodeIndex_t

[jakub at gcc dot gnu dot org]

------- Comment #4 from jakub at gcc dot gnu dot org  2009-01-02 20:09 -------
Testing a fix.


-- 

jakub at gcc dot gnu dot org changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
         AssignedTo|unassigned at gcc dot gnu   |jakub at gcc dot gnu dot org
                   |dot org                     |
             Status|NEW                         |ASSIGNED
   Last reconfirmed|2009-01-02 18:38:20         |2009-01-02 20:09:12
               date|                            |


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

[Bug c++/38705] [4.4 Regression] ICE: canonical types differ for identical types const int and const AlpsNodeIndex_t

[jakub at gcc dot gnu dot org]

------- Comment #5 from jakub at gcc dot gnu dot org  2009-01-03 00:56 -------
Subject: Bug 38705

Author: jakub
Date: Sat Jan  3 00:54:40 2009
New Revision: 143029

URL: http://gcc.gnu.org/viewcvs?root=gcc&view=rev&rev=143029
Log:
        PR c++/38705
        * builtins.c (fold_builtin_memory_op): Give up if either operand
        is volatile.  Set srctype or desttype to non-qualified version
        of the other type.

        * g++.dg/torture/pr38705.C: New test.

Added:
    trunk/gcc/testsuite/g++.dg/torture/pr38705.C
Modified:
    trunk/gcc/ChangeLog
    trunk/gcc/builtins.c
    trunk/gcc/testsuite/ChangeLog


-- 


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

[Bug c++/38705] [4.4 Regression] ICE: canonical types differ for identical types const int and const AlpsNodeIndex_t

[jakub at gcc dot gnu dot org]

------- Comment #6 from jakub at gcc dot gnu dot org  2009-01-03 00:56 -------
Fixed.


-- 

jakub at gcc dot gnu dot org changed:

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


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

[Bug c++/38705] [4.4 Regression] ICE: canonical types differ for identical types const int and const AlpsNodeIndex_t

[steven at gcc dot gnu dot org]

------- Comment #7 from steven at gcc dot gnu dot org  2009-01-03 10:24 -------
Reviewer said: "So, this is ok with or without the volatile restriction." (see
http://gcc.gnu.org/ml/gcc-patches/2009-01/msg00070.html).  The committed patch
still seems to have this restriction...?


-- 


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

[Bug c++/38705] [4.4 Regression] ICE: canonical types differ for identical types const int and const AlpsNodeIndex_t

[jakub at gcc dot gnu dot org]

------- Comment #8 from jakub at gcc dot gnu dot org  2009-01-03 11:28 -------
Yes.  Just removing the restriction is wrong, because then volatile from the
wrong side of the copy would be used.  So extra code would be needed to handle
that, and I really don't think it is worth it.


-- 


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