[Bug bootstrap/54478] New: Fails to bootstrap with GCC 4.3.6

[Bug bootstrap/54478] New: Fails to bootstrap with GCC 4.3.6

[rguenth at gcc dot gnu.org]

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

             Bug #: 54478
           Summary: Fails to bootstrap with GCC 4.3.6
    Classification: Unclassified
           Product: gcc
           Version: 4.8.0
            Status: UNCONFIRMED
          Keywords: build
          Severity: normal
          Priority: P3
         Component: bootstrap
        AssignedTo: unassigned@gcc.gnu.org
        ReportedBy: rguenth@gcc.gnu.org


Recent vec.h changes make bootstrap fail with 4.3.6:

g++ -c   -g -DIN_GCC   -fno-exceptions -fno-rtti -W -Wall -Wwrite-strings
-Wcast-qual -Wmissing-format-attribute -pedantic -Wno-long-long
-Wno-variadic-macros -Wno-overlength-strings -fno-common  -DHAVE_CONFIG_H
-DGENERATOR_FILE -I. -Ibuild -I/space/rguenther/src/svn/trunk/gcc
-I/space/rguenther/src/svn/trunk/gcc/build
-I/space/rguenther/src/svn/trunk/gcc/../include
-I/space/rguenther/src/svn/trunk/gcc/../libcpp/include 
-I/space/rguenther/src/svn/trunk/gcc/../libdecnumber
-I/space/rguenther/src/svn/trunk/gcc/../libdecnumber/bid -I../libdecnumber
-DCLOOG_INT_GMP    \
>                 -o build/read-rtl.o /space/rguenther/src/svn/trunk/gcc/read-rtl.c -save-temps
/space/rguenther/src/svn/trunk/gcc/vec.h: In static member function 'static T&
vec_t<T>::safe_push(vec_t<T>**, T, const char*, unsigned int, const char*)
[with vec_allocation_t A = heap, T = mapping*]':
/space/rguenther/src/svn/trunk/gcc/read-rtl.c:395:   instantiated from here
/space/rguenther/src/svn/trunk/gcc/vec.h:845: warning: left-hand operand of
comma has no effect
...
/space/rguenther/src/svn/trunk/gcc/vec.h:845: error: no match for 'operator<'
in 'vec_t::reserve [with vec_allocation_t A = A, T = mapping*] < heap'
/space/rguenther/src/svn/trunk/gcc/vec.h: In static member function 'static T*
vec_t<T>::safe_push(vec_t<T>**, const T*, const char*, unsigned int, const
char*) [with vec_allocation_t A = heap, T = iterator_use]':
...

seems to parse vec<T>::safe_push<A> () not as call.

[Bug bootstrap/54478] Fails to bootstrap with GCC 4.3.6

[rguenth at gcc dot gnu.org]

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

--- Comment #1 from Richard Guenther <rguenth at gcc dot gnu.org> 2012-09-04 13:57:52 UTC ---
Created attachment 28129
  --> http://gcc.gnu.org/bugzilla/attachment.cgi?id=28129
preprocessed source

[Bug bootstrap/54478] [4.8 Regression] Fails to bootstrap with GCC 4.3.6

[rguenth at gcc dot gnu.org]

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

Richard Guenther <rguenth at gcc dot gnu.org> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
      Known to work|                            |4.4.6
   Target Milestone|---                         |4.8.0
            Summary|Fails to bootstrap with GCC |[4.8 Regression] Fails to
                   |4.3.6                       |bootstrap with GCC 4.3.6
      Known to fail|                            |4.1.2, 4.3.6

--- Comment #2 from Richard Guenther <rguenth at gcc dot gnu.org> 2012-09-04 13:59:41 UTC ---
Not sure if non-GCC compilers are affected.

[Bug bootstrap/54478] [4.8 Regression] Fails to bootstrap with GCC 4.3.6

[dnovillo at gcc dot gnu.org]

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

Diego Novillo <dnovillo at gcc dot gnu.org> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
             Status|UNCONFIRMED                 |ASSIGNED
   Last reconfirmed|                            |2012-09-04
                 CC|                            |dnovillo at gcc dot gnu.org
         AssignedTo|unassigned at gcc dot       |dnovillo at gcc dot gnu.org
                   |gnu.org                     |
     Ever Confirmed|0                           |1

--- Comment #3 from Diego Novillo <dnovillo at gcc dot gnu.org> 2012-09-04 16:04:04 UTC ---
Working on a fix.

[Bug bootstrap/54478] [4.8 Regression] Fails to bootstrap with GCC 4.3.6

[dnovillo at gcc dot gnu.org]

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

--- Comment #4 from Diego Novillo <dnovillo at gcc dot gnu.org> 2012-09-04 18:33:34 UTC ---
Author: dnovillo
Date: Tue Sep  4 18:33:29 2012
New Revision: 190943

URL: http://gcc.gnu.org/viewcvs?root=gcc&view=rev&rev=190943
Log:
This patch works around a parsing problem with g++ 4.3.  The parser is
failing to lookup calls to the template function reserve when called
from other member functions:

vec_t<T>::reserve<A> (...)

The parser thinks that the '<' in reserve<A> is a less-than operation.
This problem does not happen after 4.3.

This code is going to change significantly, so this won't be needed
soon.

Tested on x86_64 with g++ 4.3 and g++ 4.6.

    PR bootstrap/54478
    * vec.h (vec_t::alloc): Remove explicit type specification
    in call to reserve.
    (vec_t::copy): Likewise.
    (vec_t::reserve): Likewise.
    (vec_t::reserve_exact): Likewise.
    (vec_t::safe_splice): Likewise.
    (vec_t::safe_push): Likewise.
    (vec_t::safe_grow): Likewise.
    (vec_t::safe_grow_cleared): Likewise.
    (vec_t::safe_insert): Likewise.

Modified:
    trunk/gcc/ChangeLog
    trunk/gcc/vec.h

[Bug bootstrap/54478] [4.8 Regression] Fails to bootstrap with GCC 4.3.6

[dnovillo at gcc dot gnu.org]

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

Diego Novillo <dnovillo at gcc dot gnu.org> changed:

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

--- Comment #5 from Diego Novillo <dnovillo at gcc dot gnu.org> 2012-09-04 18:34:14 UTC ---
Fixed.

[Bug bootstrap/54478] [4.8 Regression] Fails to bootstrap with GCC 4.3.6

[dnovillo at gcc dot gnu.org]

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

Diego Novillo <dnovillo at gcc dot gnu.org> changed:

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

--- Comment #4 from Diego Novillo <dnovillo at gcc dot gnu.org> 2012-09-04 18:33:34 UTC ---
Author: dnovillo
Date: Tue Sep  4 18:33:29 2012
New Revision: 190943

URL: http://gcc.gnu.org/viewcvs?root=gcc&view=rev&rev=190943
Log:
This patch works around a parsing problem with g++ 4.3.  The parser is
failing to lookup calls to the template function reserve when called
from other member functions:

vec_t<T>::reserve<A> (...)

The parser thinks that the '<' in reserve<A> is a less-than operation.
This problem does not happen after 4.3.

This code is going to change significantly, so this won't be needed
soon.

Tested on x86_64 with g++ 4.3 and g++ 4.6.

    PR bootstrap/54478
    * vec.h (vec_t::alloc): Remove explicit type specification
    in call to reserve.
    (vec_t::copy): Likewise.
    (vec_t::reserve): Likewise.
    (vec_t::reserve_exact): Likewise.
    (vec_t::safe_splice): Likewise.
    (vec_t::safe_push): Likewise.
    (vec_t::safe_grow): Likewise.
    (vec_t::safe_grow_cleared): Likewise.
    (vec_t::safe_insert): Likewise.

Modified:
    trunk/gcc/ChangeLog
    trunk/gcc/vec.h

--- Comment #5 from Diego Novillo <dnovillo at gcc dot gnu.org> 2012-09-04 18:34:14 UTC ---
Fixed.