Re: c++/7807: g++ 3.2 Fails to compile legal code that compiled OK with g++ 3.1

Re: c++/7807: g++ 3.2 Fails to compile legal code that compiled OK with g++ 3.1

[nathan]

Synopsis: g++ 3.2 Fails to compile legal code that compiled OK with g++ 3.1

State-Changed-From-To: open->analyzed
State-Changed-By: nathan
State-Changed-When: Fri Sep 13 13:35:35 2002
State-Changed-Why:
    confirmed as a regression

http://gcc.gnu.org/cgi-bin/gnatsweb.pl?cmd=view%20audit-trail&database=gcc&pr=7807

Re: c++/7807: g++ 3.2 Fails to compile legal code that compiled OK with g++ 3.1

[mmitchel]

Synopsis: g++ 3.2 Fails to compile legal code that compiled OK with g++ 3.1

State-Changed-From-To: analyzed->closed
State-Changed-By: mmitchel
State-Changed-When: Wed Oct 16 12:41:17 2002
State-Changed-Why:
    Fixed in GCC 3.2.1.

http://gcc.gnu.org/cgi-bin/gnatsweb.pl?cmd=view%20audit-trail&database=gcc&pr=7807

Re: c++/7807: g++ 3.2 Fails to compile legal code that compiled OK with g++ 3.1

[Pop Sébastian]

The following reply was made to PR c++/7807; it has been noted by GNATS.

From: =?iso-8859-1?Q?Pop_S=E9bastian?= <pop@gauvain.u-strasbg.fr>
To: nathan@gcc.gnu.org, anthony@anthonyw.cjb.net, gcc-bugs@gcc.gnu.org,
	gcc-prs@gcc.gnu.org, nobody@gcc.gnu.org, gcc-gnats@gcc.gnu.org
Cc: aoliva@redhat.com
Subject: Re: c++/7807: g++ 3.2 Fails to compile legal code that compiled OK with g++ 3.1
Date: Sun, 29 Sep 2002 07:50:28 +0200

 On Fri, Sep 13, 2002 at 08:35:36PM -0000, nathan@gcc.gnu.org wrote:
 > Synopsis: g++ 3.2 Fails to compile legal code that compiled OK with g++ 3.1
 > 
 > State-Changed-From-To: open->analyzed
 > State-Changed-By: nathan
 > State-Changed-When: Fri Sep 13 13:35:35 2002
 > State-Changed-Why:
 >     confirmed as a regression
 > 
 > http://gcc.gnu.org/cgi-bin/gnatsweb.pl?cmd=view%20audit-trail&database=gcc&pr=7807
 
 The bug was introduced with the following patch:
 
 2002-05-14  Alexandre Oliva  <aoliva@redhat.com>
 
 Patch for C++ uninitialized pointer-to-member value
 http://gcc.gnu.org/ml/gcc-patches/2002-05/msg01167.html
 
 
 Here is a simplified version of the original bug-example on which I worked:
 
 class RM
 {
   class U;
   U* (U::* s2);
 };
 
 class OP
 {
   int& f;
   RM pb;
   
 public:
   OP (int f_): f (f_) {}
 };
 
 int i=9;
 OP a(i);
 
 
 
 The following patch is a workaround that solves bug-reports 7648 and 7807.
 Alexandre, could you analyze further this problem?
 
 Thanks,
 Sebastian
 
 
 
 Index: decl.c
 ===================================================================
 RCS file: /cvsroot/gcc/gcc/gcc/cp/decl.c,v
 retrieving revision 1.936
 diff -d -u -p -r1.936 decl.c
 --- decl.c	21 Sep 2002 12:51:53 -0000	1.936
 +++ decl.c	29 Sep 2002 05:45:34 -0000
 @@ -7830,7 +7830,7 @@ check_initializer (decl, init)
      }
    else if (!DECL_EXTERNAL (decl) && !zero_init_p (type))
      {
 -      force_store_init_value (decl, build_forced_zero_init (type));
 +//      force_store_init_value (decl, build_forced_zero_init (type));
  
        if (init)
  	goto process_init;
 Index: typeck.c
 ===================================================================
 RCS file: /cvsroot/gcc/gcc/gcc/cp/typeck.c,v
 retrieving revision 1.429
 diff -d -u -p -r1.429 typeck.c
 --- typeck.c	21 Sep 2002 12:51:56 -0000	1.429
 +++ typeck.c	29 Sep 2002 05:45:35 -0000
 @@ -5838,8 +5838,12 @@ convert_for_assignment (type, rhs, errty
    rhstype = TREE_TYPE (rhs);
    coder = TREE_CODE (rhstype);
  
 -  if (rhs == error_mark_node || rhstype == error_mark_node)
 +  if (rhs == error_mark_node)
 +    return error_mark_node;
 +  
 +  if (rhstype == error_mark_node)
      return error_mark_node;
 +      
    if (TREE_CODE (rhs) == TREE_LIST && TREE_VALUE (rhs) == error_mark_node)
      return error_mark_node;
  

Re: c++/7807: g++ 3.2 Fails to compile legal code that compiled OK with g++ 3.1

[Andris Pavenis]

The following reply was made to PR c++/7807; it has been noted by GNATS.

From: Andris Pavenis <pavenis@lanet.lv>
To: gcc-gnats@gcc.gnu.org, gcc-prs@gcc.gnu.org,
	anthony@anthonyw.cjb.net, gcc-bugs@gcc.gnu.org, nobody@gcc.gnu.org
Cc:  
Subject: Re: c++/7807: g++ 3.2 Fails to compile legal code that compiled OK
 with g++ 3.1
Date: Mon, 02 Sep 2002 17:56:30 +0300

 Reproduced under Linux.
 
 Looks the same as PR#7648 I reported about 2 weeks ago (for i386-slackware-linux and
 i586-pc-msdosdjgpp).
 
 http://gcc.gnu.org/cgi-bin/gnatsweb.pl?cmd=view%20audit-trail&database=gcc&pr=7648
 
 Andris
 
 
 
 http://gcc.gnu.org/cgi-bin/gnatsweb.pl?cmd=view%20audit-trail&database=gcc&pr=7807
 
 
 

c++/7807: g++ 3.2 Fails to compile legal code that compiled OK with g++ 3.1

[anthony]

>Number:         7807
>Category:       c++
>Synopsis:       g++ 3.2 Fails to compile legal code that compiled OK with g++ 3.1
>Confidential:   no
>Severity:       critical
>Priority:       medium
>Responsible:    unassigned
>State:          open
>Class:          sw-bug
>Submitter-Id:   net
>Arrival-Date:   Mon Sep 02 02:36:01 PDT 2002
>Closed-Date:
>Last-Modified:
>Originator:     Anthony Williams
>Release:        g++-3.2-mingw32
>Organization:
>Environment:
g++-3.2-mingw32
mingw-runtime 2.1
w32api 1.5
>Description:
The attached file produces errors about "uninitialized const member"s on g++ 3.2, but doesn't with g++ 3.1, nor if you remove the member function pointer from the RawMem union.
>How-To-Repeat:
Compile the attached file
>Fix:

>Release-Note:
>Audit-Trail:
>Unformatted:
----gnatsweb-attachment----
Content-Type: application/octet-stream; name="cpp_regress.ii"
Content-Transfer-Encoding: base64
Content-Disposition: attachment; filename="cpp_regress.ii"

IyAxICJjcHBfcmVncmVzcy5jcHAiDQojIDEgIjxidWlsdC1pbj4iDQojIDEgIjxjb21tYW5kIGxp
bmU+Ig0KIyAxICJjcHBfcmVncmVzcy5jcHAiDQppbnQgaTEsaTI7DQoNCm5hbWVzcGFjZSB1dGls
cw0Kew0KICAgIGNsYXNzIFVua25vd247DQoNCiAgICB0ZW1wbGF0ZTx0eXBlbmFtZSBUPg0KICAg
IHVuaW9uIFJhd01lbQ0KICAgIHsNCiAgICAgICAgY2hhciBkYXRhW3NpemVvZihUKV07DQogICAg
ICAgIFVua25vd24qIChVbmtub3duOjoqIHNtZHApOw0KICAgIH07DQp9DQoNCg0KDQogICAgdGVt
cGxhdGU8dHlwZW5hbWUgVCx0eXBlbmFtZSBVPg0KICAgIHN0cnVjdCBPd25pbmdSZWZQYWlyDQog
ICAgew0KICAgIHB1YmxpYzoNCiAgICAgICAgVCYgZmlyc3Q7DQogICAgICAgIFUmIHNlY29uZDsN
Cg0KICAgIHByaXZhdGU6DQogICAgICAgIHV0aWxzOjpSYXdNZW08aW50PiBwYWlyQnVmOw0KICAg
ICAgICBjb25zdCBib29sIG93bnNGbGFnOw0KDQogICAgcHVibGljOg0KDQogICAgICAgIE93bmlu
Z1JlZlBhaXIoY29uc3QgT3duaW5nUmVmUGFpciYgb3RoZXIpOg0KICAgICAgICAgICAgZmlyc3Qo
aTEpLA0KICAgICAgICAgICAgc2Vjb25kKGkyKSwNCiAgICAgICAgICAgIG93bnNGbGFnKHRydWUp
DQogICAgICAgIHsNCiAgICAgICAgfQ0KDQoNCg0KICAgICAgICB0ZW1wbGF0ZTx0eXBlbmFtZSBW
YWwxLHR5cGVuYW1lIFZhbDI+DQogICAgICAgIE93bmluZ1JlZlBhaXIoY29uc3QgVmFsMSYgZmly
c3RfLGNvbnN0IFZhbDImIHNlY29uZF8pOg0KICAgICAgICAgICAgZmlyc3QoaTEpLA0KICAgICAg
ICAgICAgc2Vjb25kKGkyKSwNCiAgICAgICAgICAgIG93bnNGbGFnKHRydWUpDQogICAgICAgIHsN
CiAgICAgICAgfQ0KDQoNCg0KICAgIH07DQoNCg0KT3duaW5nUmVmUGFpcjxpbnQsaW50PiBhKGkx
LGkyKTsNCk93bmluZ1JlZlBhaXI8aW50LGludD4gYj1hOw0K