[Bug c++/32304] New: ICE in set_mem_attributes_minus_bitpos

[Bug c++/32304] New: ICE in set_mem_attributes_minus_bitpos

[wouter dot vermaelen at scarlet dot be]

[-- Warning: decoded text below may be mangled, UTF-8 assumed --]
[-- Attachment #1: Type: text/plain, Size: 617 bytes --]

> cat bug.ii
struct S {
        S() {}
};
S f() {
        static S s;
        return s;
}

> g++ -O bug.ii
bug2.ii: In function ‘S f()’:
bug2.ii:6: internal compiler error: in set_mem_attributes_minus_bitpos, at
emit-rtl.c:1573


-- 
           Summary: ICE in set_mem_attributes_minus_bitpos
           Product: gcc
           Version: 4.3.0
            Status: UNCONFIRMED
          Severity: normal
          Priority: P3
         Component: c++
        AssignedTo: unassigned at gcc dot gnu dot org
        ReportedBy: wouter dot vermaelen at scarlet dot be


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

[Bug middle-end/32304] [4.3 Regression] ICE in set_mem_attributes_minus_bitpos

[pinskia at gcc dot gnu dot org]

------- Comment #1 from pinskia at gcc dot gnu dot org  2007-06-12 14:20 -------
readonly is set on this decl which is wrong.

I think I know what is wrong.
ipa-reference.c sets TREE_READONLY on the decl when it really should not be
(even though it is readonly now).


-- 

pinskia at gcc dot gnu dot org changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
                 CC|                            |pinskia at gcc dot gnu dot
                   |                            |org
             Status|UNCONFIRMED                 |NEW
          Component|c++                         |middle-end
     Ever Confirmed|0                           |1
           Keywords|                            |ice-on-valid-code
   Last reconfirmed|0000-00-00 00:00:00         |2007-06-12 14:20:18
               date|                            |
            Summary|ICE in                      |[4.3 Regression] ICE in
                   |set_mem_attributes_minus_bit|set_mem_attributes_minus_bit
                   |pos                         |pos
   Target Milestone|---                         |4.3.0


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

[Bug middle-end/32304] [4.3 Regression] ICE in set_mem_attributes_minus_bitpos

[pinskia at gcc dot gnu dot org]

------- Comment #2 from pinskia at gcc dot gnu dot org  2007-06-12 14:22 -------
*** Bug 31685 has been marked as a duplicate of this bug. ***


-- 

pinskia at gcc dot gnu dot org changed:

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


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

[Bug middle-end/32304] [4.3 Regression] ICE in set_mem_attributes_minus_bitpos

[pinskia at gcc dot gnu dot org]

------- Comment #3 from pinskia at gcc dot gnu dot org  2007-06-12 18:16 -------
The patch which fixes the problem:
Index: ipa-reference.c
===================================================================
--- ipa-reference.c     (revision 125637)
+++ ipa-reference.c     (working copy)
@@ -269,6 +269,10 @@
   if (DECL_EXTERNAL (t) || TREE_PUBLIC (t))
     return false;

+  /* We cannot touch decl's whos types need constructing. */
+  if (TYPE_NEEDS_CONSTRUCTING (TREE_TYPE (t)))
+    return false;
+
   /* This is a variable we care about.  Check if we have seen it
      before, and if not add it the set of variables we care about.  */
   if (!bitmap_bit_p (all_module_statics, DECL_UID (t)))


-- 

pinskia at gcc dot gnu dot org changed:

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


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

[Bug middle-end/32304] [4.3 Regression] ICE in set_mem_attributes_minus_bitpos

[mmitchel at gcc dot gnu dot org]

-- 

mmitchel at gcc dot gnu dot org changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
           Priority|P3                          |P1


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

[Bug middle-end/32304] [4.3 Regression] ICE in set_mem_attributes_minus_bitpos

[pinskia at gcc dot gnu dot org]

------- Comment #4 from pinskia at gcc dot gnu dot org  2007-08-04 00:06 -------
Subject: Bug 32304

Author: pinskia
Date: Sat Aug  4 00:05:56 2007
New Revision: 127191

URL: http://gcc.gnu.org/viewcvs?root=gcc&view=rev&rev=127191
Log:
2007-08-03  Andrew Pinski  <andrew_pinski@playstation.sony.com>

        PR middle-end/32304
        * ipa-reference.c (has_proper_scope_for_analysis): Return false when
        the decl's type has TYPE_NEEDS_CONSTRUCTING set.

2007-08-03  Andrew Pinski  <andrew_pinski@playstation.sony.com>

        PR middle-end/32304
        * g++.dg/torture/pr32304.C: New testcase.


Added:
    trunk/gcc/testsuite/g++.dg/torture/pr32304.C
Modified:
    trunk/gcc/ChangeLog
    trunk/gcc/ipa-reference.c
    trunk/gcc/testsuite/ChangeLog


-- 


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

[Bug middle-end/32304] [4.3 Regression] ICE in set_mem_attributes_minus_bitpos

[pinskia at gcc dot gnu dot org]

------- Comment #5 from pinskia at gcc dot gnu dot org  2007-08-04 00:06 -------
Fixed.


-- 

pinskia at gcc dot gnu dot org changed:

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


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