[Bug c++/13592] New: C++ parser regression

[Bug c++/13592] New: C++ parser regression

[rwgk at yahoo dot com]

gcc snapshot from 2003/12/30, ca. 7:40am PST
Configured with: ../gcc/configure --prefix=/usr/local_cci/gcc_cvs_2003_12_30 --
enable-languages=c,c++
Redhat Linux 8.0

gcc 3.4.0 fails to compile this piece of code:

        result[ir] = boost::lcm(
          result[ir], norm_denominator(r_(ir, ic), grid[ic]));

However, it works with this trivial modification:

        result[ir] = boost::lcm(
          result[ir], norm_denominator(this->r_(ir, ic), grid[ic]));

The original code works with many other compilers incl. earlier gcc's. Note 
that r_ is a member of the rt_mx class and *not* inherited. I will attach a 
preprocessed file that can be used to reproduce the problem and the workaround:

g++ -fPIC -ftemplate-depth-120 -w -DNDEBUG -O3 -DBOOST_PYTHON_MAX_BASES=2 -
Iinclude -I/net/worm/scratch1/rwgk/rc1310/include -
I/net/worm/scratch1/rwgk/rc1310/cctbx/include -
I/net/worm/scratch1/rwgk/rc1310/scitbx/include -
I/net/worm/scratch1/rwgk/rc1310/boost -I/usr/include/python2.2 -
E /net/worm/scratch1/rwgk/rc1310/cctbx/adptbx/boost_python/adptbx_ext.cpp > 
adptbx_ext_pp.cpp

g++ -fPIC -ftemplate-depth-120 -w -O3 -c adptbx_ext_pp.cpp

In file included 
from /net/worm/scratch1/rwgk/rc1310/cctbx/include/cctbx/sgtbx/change_of_basis_op
.h:16,
                 
from /net/worm/scratch1/rwgk/rc1310/cctbx/include/cctbx/sgtbx/tr_group.h:17,
                 
from /net/worm/scratch1/rwgk/rc1310/cctbx/include/cctbx/sgtbx/space_group.h:17,
                 
from /net/worm/scratch1/rwgk/rc1310/cctbx/include/cctbx/sgtbx/site_symmetry.h:14
,
                 
from /net/worm/scratch1/rwgk/rc1310/cctbx/include/cctbx/xray/scatterer.h:4,
                 
from /net/worm/scratch1/rwgk/rc1310/cctbx/include/cctbx/boost_python/flex_fwd.h:
10,
                 
from /net/worm/scratch1/rwgk/rc1310/cctbx/adptbx/boost_python/adptbx_ext.cpp:1:
/net/worm/scratch1/rwgk/rc1310/cctbx/include/cctbx/sgtbx/rt_mx.h: In member 
function `GridTupleType cctbx::sgtbx::rt_mx::refine_gridding(const 
GridTupleType&) const':
/net/worm/scratch1/rwgk/rc1310/cctbx/include/cctbx/sgtbx/rt_mx.h:414: error: 
call to non-function `cctbx::sgtbx::rt_mx::r_'

diff adptbx_ext_pp.cpp adptbx_ext_pp_patched.cpp
47390c47390
<           result[ir], norm_denominator(r_(ir, ic), grid[ic]));
---
>           result[ir], norm_denominator(this->r_(ir, ic), grid[ic]));

g++ -fPIC -ftemplate-depth-120 -w -O3 -c adptbx_ext_pp_patched.cpp

Succeeds.

-- 
           Summary: C++ parser regression
           Product: gcc
           Version: 3.4.0
            Status: UNCONFIRMED
          Severity: normal
          Priority: P2
         Component: c++
        AssignedTo: unassigned at gcc dot gnu dot org
        ReportedBy: rwgk at yahoo dot com
                CC: gcc-bugs at gcc dot gnu dot org
 GCC build triplet: g++ (GCC) 3.4.0 20031230 (experimental)
  GCC host triplet: i686-gnu-linux
GCC target triplet: i686-gnu-linux


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

[Bug c++/13592] C++ parser regression

[rwgk at yahoo dot com]

------- Additional Comments From rwgk at yahoo dot com  2004-01-06 23:37 -------
Created an attachment (id=5424)
 --> (http://gcc.gnu.org/bugzilla/attachment.cgi?id=5424&action=view)
Preprocessed source code

Also available here:

http://cci.lbl.gov/~rwgk/bugs/gcc340/adptbx_ext_pp.cpp

http://cci.lbl.gov/~rwgk/bugs/gcc340/adptbx_ext_pp_patched.cpp


-- 


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

[Bug c++/13592] [3.4 Regression] C++ parser regression

[pinskia at gcc dot gnu dot org]

------- Additional Comments From pinskia at gcc dot gnu dot org  2004-01-07 01:22 -------
Confirmed, reduced source:
struct rot_mx
{
  int const& operator()(int r, int c) const;
  int& operator()(int r, int c);
};

struct rt_mx
{
    template <typename GridTupleType>
     GridTupleType refine_gridding(GridTupleType const& grid) const;
    rot_mx r_;
};

template <typename GridTupleType>
GridTupleType
rt_mx::refine_gridding(GridTupleType const& grid) const
{
  GridTupleType result;
  r_(0, 0);
  return result;
}

-- 
           What    |Removed                     |Added
----------------------------------------------------------------------------
           Severity|normal                      |critical
             Status|UNCONFIRMED                 |NEW
     Ever Confirmed|                            |1
  GCC build triplet|g++ (GCC) 3.4.0 20031230    |
                   |(experimental)              |
   GCC host triplet|i686-gnu-linux              |
 GCC target triplet|i686-gnu-linux              |
           Keywords|                            |rejects-valid
   Last reconfirmed|0000-00-00 00:00:00         |2004-01-07 01:22:21
               date|                            |
            Summary|C++ parser regression       |[3.4 Regression] C++ parser
                   |                            |regression
   Target Milestone|---                         |3.4.0


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

[Bug c++/13592] [3.4 Regression] C++ parser regression

[pinskia at gcc dot gnu dot org]

------- Additional Comments From pinskia at gcc dot gnu dot org  2004-01-07 01:22 -------
>From Phil's regression hunter: Search converges between 2003-07-17-trunk (#346) and 2003-07
-19-trunk (#347).

-- 


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

[Bug c++/13592] [3.4 Regression] C++ parser regression

[pinskia at gcc dot gnu dot org]

------- Additional Comments From pinskia at gcc dot gnu dot org  2004-01-07 05:21 -------
Nathan Could you look at this bug, it looks like your patch causes it:

2003-07-18  Nathan Sidwell  <nathan@codesourcery.com>

        * cp-tree.h (finish_non_static_data_member): Add object param.
        * method.c (hack_identifier): Adjust.
        * pt.c (tsubst_copy_and_build) <COMPONENT_REF case>: Don't search
        again for a FIELD_DECL.
        * semantics.c (finish_non_static_data_member): Add object
        parameter. Always save the DECL in the COMPONENT_REF.
        * call.c (resolve_scoped_fn_name): Adjust.

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


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

[Bug c++/13592] [3.4 Regression] C++ parser regression

[bangerth at dealii dot org]

------- Additional Comments From bangerth at dealii dot org  2004-01-07 14:07 -------
Here is something that's more in our standard format: 
-------------------- 
struct S { 
    void operator()(int); 
}; 
 
struct A { 
    template <typename> void foo(); 
    S s; 
}; 
 
template <typename> void A::foo() { 
  s(0); 
} 
-------------------------- 
 
g/x> /home/bangerth/bin/gcc-3.3.3-pre/bin/c++ -c x.cc 
g/x> /home/bangerth/bin/gcc-3.4-pre/bin/c++ -c x.cc 
x.cc: In member function `void A::foo()': 
x.cc:11: error: call to non-function `A::s' 
 
W. 

-- 


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

[Bug c++/13592] [3.4 Regression] C++ parser regression

[steven at gcc dot gnu dot org]

------- Additional Comments From steven at gcc dot gnu dot org  2004-01-12 19:06 -------
For the sake of completeness, this is the patch that causes this bug: 
http://gcc.gnu.org/ml/gcc-patches/2003-07/msg01910.html 

-- 


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

[Bug c++/13592] [3.4/3.5 Regression] C++ parser regression

[cvs-commit at gcc dot gnu dot org]

------- Additional Comments From cvs-commit at gcc dot gnu dot org  2004-01-19 20:33 -------
Subject: Bug 13592

CVSROOT:	/cvs/gcc
Module name:	gcc
Changes by:	mmitchel@gcc.gnu.org	2004-01-19 20:33:30

Modified files:
	gcc/cp         : ChangeLog call.c class.c cp-tree.h init.c 
	                 parser.c pt.c semantics.c 
	gcc/testsuite  : ChangeLog 
Added files:
	gcc/testsuite/g++.dg/template: call2.C 

Log message:
	PR c++/13592
	* call.c (build_field_call): Remove.
	(n_build_method_call): Likewise.
	(build_method_call): Likewise.
	(build_new_method_call): Do not call build_field_call.
	* class.c (n_build_method_call): Remove.
	(print_class_statistics): Do not print it.
	* cp-tree.h (build_method_call): Remove declaration.
	(finish_object_call_expr): Likewise.
	(build_new_1): Do not use build_method_call.
	* parser.c (cp_parser_postfix_expression): Use finish_call_expr
	when the function appearing on the right-hand-side of "." or "->"
	is not actually a function.
	* pt.c (tsubst_copy_and_build): Likewise.
	* semantics.c (finish_object_call_expr): Remove.
	
	PR c++/13592
	* g++.dg/template/call2.C: New test.

Patches:
http://gcc.gnu.org/cgi-bin/cvsweb.cgi/gcc/gcc/cp/ChangeLog.diff?cvsroot=gcc&r1=1.3897&r2=1.3898
http://gcc.gnu.org/cgi-bin/cvsweb.cgi/gcc/gcc/cp/call.c.diff?cvsroot=gcc&r1=1.452&r2=1.453
http://gcc.gnu.org/cgi-bin/cvsweb.cgi/gcc/gcc/cp/class.c.diff?cvsroot=gcc&r1=1.595&r2=1.596
http://gcc.gnu.org/cgi-bin/cvsweb.cgi/gcc/gcc/cp/cp-tree.h.diff?cvsroot=gcc&r1=1.946&r2=1.947
http://gcc.gnu.org/cgi-bin/cvsweb.cgi/gcc/gcc/cp/init.c.diff?cvsroot=gcc&r1=1.356&r2=1.357
http://gcc.gnu.org/cgi-bin/cvsweb.cgi/gcc/gcc/cp/parser.c.diff?cvsroot=gcc&r1=1.157&r2=1.158
http://gcc.gnu.org/cgi-bin/cvsweb.cgi/gcc/gcc/cp/pt.c.diff?cvsroot=gcc&r1=1.817&r2=1.818
http://gcc.gnu.org/cgi-bin/cvsweb.cgi/gcc/gcc/cp/semantics.c.diff?cvsroot=gcc&r1=1.381&r2=1.382
http://gcc.gnu.org/cgi-bin/cvsweb.cgi/gcc/gcc/testsuite/ChangeLog.diff?cvsroot=gcc&r1=1.3398&r2=1.3399
http://gcc.gnu.org/cgi-bin/cvsweb.cgi/gcc/gcc/testsuite/g++.dg/template/call2.C.diff?cvsroot=gcc&r1=NONE&r2=1.1



-- 


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

[Bug c++/13592] [3.4/3.5 Regression] C++ parser regression

[cvs-commit at gcc dot gnu dot org]

------- Additional Comments From cvs-commit at gcc dot gnu dot org  2004-01-19 20:39 -------
Subject: Bug 13592

CVSROOT:	/cvs/gcc
Module name:	gcc
Branch: 	gcc-3_4-branch
Changes by:	mmitchel@gcc.gnu.org	2004-01-19 20:39:40

Modified files:
	gcc/cp         : ChangeLog call.c class.c cp-tree.h init.c 
	                 parser.c pt.c semantics.c 
	gcc/testsuite  : ChangeLog 
Added files:
	gcc/testsuite/g++.dg/template: call2.C 

Log message:
	PR c++/13592
	* call.c (build_field_call): Remove.
	(n_build_method_call): Likewise.
	(build_method_call): Likewise.
	(build_new_method_call): Do not call build_field_call.
	* class.c (n_build_method_call): Remove.
	(print_class_statistics): Do not print it.
	* cp-tree.h (build_method_call): Remove declaration.
	(finish_object_call_expr): Likewise.
	(build_new_1): Do not use build_method_call.
	* parser.c (cp_parser_postfix_expression): Use finish_call_expr
	when the function appearing on the right-hand-side of "." or "->"
	is not actually a function.
	* pt.c (tsubst_copy_and_build): Likewise.
	* semantics.c (finish_object_call_expr): Remove.
	
	PR c++/13592
	* g++.dg/template/call2.C: New test.

Patches:
http://gcc.gnu.org/cgi-bin/cvsweb.cgi/gcc/gcc/cp/ChangeLog.diff?cvsroot=gcc&only_with_tag=gcc-3_4-branch&r1=1.3892.2.3&r2=1.3892.2.4
http://gcc.gnu.org/cgi-bin/cvsweb.cgi/gcc/gcc/cp/call.c.diff?cvsroot=gcc&only_with_tag=gcc-3_4-branch&r1=1.452&r2=1.452.2.1
http://gcc.gnu.org/cgi-bin/cvsweb.cgi/gcc/gcc/cp/class.c.diff?cvsroot=gcc&only_with_tag=gcc-3_4-branch&r1=1.595&r2=1.595.4.1
http://gcc.gnu.org/cgi-bin/cvsweb.cgi/gcc/gcc/cp/cp-tree.h.diff?cvsroot=gcc&only_with_tag=gcc-3_4-branch&r1=1.946&r2=1.946.4.1
http://gcc.gnu.org/cgi-bin/cvsweb.cgi/gcc/gcc/cp/init.c.diff?cvsroot=gcc&only_with_tag=gcc-3_4-branch&r1=1.356&r2=1.356.2.1
http://gcc.gnu.org/cgi-bin/cvsweb.cgi/gcc/gcc/cp/parser.c.diff?cvsroot=gcc&only_with_tag=gcc-3_4-branch&r1=1.157&r2=1.157.2.1
http://gcc.gnu.org/cgi-bin/cvsweb.cgi/gcc/gcc/cp/pt.c.diff?cvsroot=gcc&only_with_tag=gcc-3_4-branch&r1=1.816.2.1&r2=1.816.2.2
http://gcc.gnu.org/cgi-bin/cvsweb.cgi/gcc/gcc/cp/semantics.c.diff?cvsroot=gcc&only_with_tag=gcc-3_4-branch&r1=1.381&r2=1.381.4.1
http://gcc.gnu.org/cgi-bin/cvsweb.cgi/gcc/gcc/testsuite/ChangeLog.diff?cvsroot=gcc&only_with_tag=gcc-3_4-branch&r1=1.3389.2.5&r2=1.3389.2.6
http://gcc.gnu.org/cgi-bin/cvsweb.cgi/gcc/gcc/testsuite/g++.dg/template/call2.C.diff?cvsroot=gcc&only_with_tag=gcc-3_4-branch&r1=NONE&r2=1.1.2.1



-- 


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

[Bug c++/13592] [3.4/3.5 Regression] C++ parser regression

[mmitchel at gcc dot gnu dot org]

------- Additional Comments From mmitchel at gcc dot gnu dot org  2004-01-19 20:41 -------
Fixed in GCC 3.4.

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


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

[Bug c++/13592] [3.4/3.5 Regression] C++ parser regression

[cvs-commit at gcc dot gnu dot org]

------- Additional Comments From cvs-commit at gcc dot gnu dot org  2004-01-19 21:24 -------
Subject: Bug 13592

CVSROOT:	/cvs/gcc
Module name:	gcc
Changes by:	mmitchel@gcc.gnu.org	2004-01-19 21:24:47

Modified files:
	gcc/testsuite  : ChangeLog 
	gcc/testsuite/g++.dg/other: error1.C 

Log message:
	PR c++/13592
	* g++.dg/other/error1.C (class foo): Tweak error message.

Patches:
http://gcc.gnu.org/cgi-bin/cvsweb.cgi/gcc/gcc/testsuite/ChangeLog.diff?cvsroot=gcc&r1=1.3399&r2=1.3400
http://gcc.gnu.org/cgi-bin/cvsweb.cgi/gcc/gcc/testsuite/g++.dg/other/error1.C.diff?cvsroot=gcc&r1=1.2&r2=1.3



-- 


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

[Bug c++/13592] [3.4/3.5 Regression] C++ parser regression

[cvs-commit at gcc dot gnu dot org]

------- Additional Comments From cvs-commit at gcc dot gnu dot org  2004-01-19 21:25 -------
Subject: Bug 13592

CVSROOT:	/cvs/gcc
Module name:	gcc
Branch: 	gcc-3_4-branch
Changes by:	mmitchel@gcc.gnu.org	2004-01-19 21:25:31

Modified files:
	gcc/testsuite/g++.dg/other: error1.C 

Log message:
	PR c++/13592
	* g++.dg/other/error1.C (class foo): Tweak error message.

Patches:
http://gcc.gnu.org/cgi-bin/cvsweb.cgi/gcc/gcc/testsuite/g++.dg/other/error1.C.diff?cvsroot=gcc&only_with_tag=gcc-3_4-branch&r1=1.2&r2=1.2.42.1



-- 


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

[Bug c++/13592] [3.4/3.5 Regression] C++ parser regression

[cvs-commit at gcc dot gnu dot org]

------- Additional Comments From cvs-commit at gcc dot gnu dot org  2004-01-19 21:26 -------
Subject: Bug 13592

CVSROOT:	/cvs/gcc
Module name:	gcc
Branch: 	gcc-3_4-branch
Changes by:	mmitchel@gcc.gnu.org	2004-01-19 21:26:13

Modified files:
	gcc/testsuite  : ChangeLog 

Log message:
	PR c++/13592
	* g++.dg/other/error1.C (class foo): Tweak error message.

Patches:
http://gcc.gnu.org/cgi-bin/cvsweb.cgi/gcc/gcc/testsuite/ChangeLog.diff?cvsroot=gcc&only_with_tag=gcc-3_4-branch&r1=1.3389.2.6&r2=1.3389.2.7



-- 


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