[Bug middle-end/44416] [4.6 regression] Failed to build 447.dealII in SPEC CPU 2006

[Bug middle-end/44416] [4.6 regression] Failed to build 447.dealII in SPEC CPU 2006

[hjl dot tools at gmail dot com]

-- 

hjl dot tools at gmail dot com changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
   Target Milestone|---                         |4.6.0


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

[Bug middle-end/44416] New: [4.6 regression] Failed to build 447.dealII in SPEC CPU 2006

[hjl dot tools at gmail dot com]

On Linux/x86-64, revision 160243 failed to
build 447.dealII in SPEC CPU 2006 at -O2 and -O3:

g++ -c -o block_vector.o -DSPEC_CPU -DNDEBUG  -Iinclude -DBOOST_DISABLE_THREADS
-Ddeal_II_dimension=3  -O3 -ffast-math -funroll-loops   -DSPEC_CPU_LP64      
block_vector.cc
In file included from include/lac/block_vector.templates.h:19:0,
                 from block_vector.cc:14:
include/lac/block_vector.h:227:17: error: 'ptrdiff_t' does not name a type
include/lac/block_vector.h:333:38: error: 'difference_type' does not name a
type
include/lac/block_vector.h:333:38: note: (perhaps 'typename
std::iterator<std::random_access_iterator_tag, typename
internal::BlockVectorIterators::Types<number,
constness>::NumberType>::difference_type' was intended)
include/lac/block_vector.h:333:54: error: ISO C++ forbids declaration of 'd'
with no type [-fpermissive]
include/lac/block_vector.h:469:9: error: 'difference_type' does not name a type
include/lac/block_vector.h:469:9: note: (perhaps 'typename
std::iterator<std::random_access_iterator_tag, typename
internal::BlockVectorIterators::Types<number,
constness>::NumberType>::difference_type' was intended)
include/lac/block_vector.h:475:9: error: 'difference_type' does not name a type
include/lac/block_vector.h:475:9: note: (perhaps 'typename
std::iterator<std::random_access_iterator_tag, typename
internal::BlockVectorIterators::Types<number,
constness>::NumberType>::difference_type' was intended)
include/lac/block_vector.h:482:36: error: 'difference_type' does not name a
type
include/lac/block_vector.h:482:36: note: (perhaps 'typename
std::iterator<std::random_access_iterator_tag, typename
internal::BlockVectorIterators::Types<number,
constness>::NumberType>::difference_type' was intended)
include/lac/block_vector.h:482:53: error: ISO C++ forbids declaration of 'd'
with no type [-fpermissive]
include/lac/block_vector.h:489:36: error: 'difference_type' does not name a
type
include/lac/block_vector.h:489:36: note: (perhaps 'typename
std::iterator<std::random_access_iterator_tag, typename
internal::BlockVectorIterators::Types<number,
constness>::NumberType>::difference_type' was intended)
include/lac/block_vector.h:489:53: error: ISO C++ forbids declaration of 'd'
with no type [-fpermissive]
include/lac/block_vector.h:496:39: error: 'difference_type' does not name a
type
include/lac/block_vector.h:496:39: note: (perhaps 'typename
std::iterator<std::random_access_iterator_tag, typename
internal::BlockVectorIterators::Types<number,
constness>::NumberType>::difference_type' was intended)
include/lac/block_vector.h:496:56: error: ISO C++ forbids declaration of 'd'
with no type [-fpermissive]
include/lac/block_vector.h:503:39: error: 'difference_type' does not name a
type
include/lac/block_vector.h:503:39: note: (perhaps 'typename
std::iterator<std::random_access_iterator_tag, typename
internal::BlockVectorIterators::Types<number,
constness>::NumberType>::difference_type' was intended)
include/lac/block_vector.h:503:56: error: ISO C++ forbids declaration of 'd'
with no type [-fpermissive]
include/lac/block_vector.h:1494:52: error: 'difference_type' does not name a
type
include/lac/block_vector.h:1494:52: note: (perhaps 'typename
std::iterator<std::random_access_iterator_tag, typename
internal::BlockVectorIterators::Types<number,
constness>::NumberType>::difference_type' was intended)
include/lac/block_vector.h:1494:68: error: ISO C++ forbids declaration of 'd'
with no type [-fpermissive]
include/lac/block_vector.h:1727:5: error: prototype for 'typename
internal::BlockVectorIterators::Iterator<number, constness>::difference_type
internal::BlockVectorIterators::Iterator<number, constness>::operator-(const
internal::BlockVectorIterators::Iterator<number, constness>&) const' does not
match any in class 'internal::BlockVectorIterators::Iterator<number,
constness>'
include/lac/block_vector.h:489:18: error: candidate is:
internal::BlockVectorIterators::Iterator<number, constness>
internal::BlockVectorIterators::Iterator<number, constness>::operator-(const
int&) const
include/lac/block_vector.h:1741:5: error: prototype for 'typename
internal::BlockVectorIterators::Iterator<number, constness>::difference_type
internal::BlockVectorIterators::Iterator<number, constness>::operator-(const
InverseConstnessIterator&) const' does not match any in class
'internal::BlockVectorIterators::Iterator<number, constness>'
include/lac/block_vector.h:489:18: error: candidate is:
internal::BlockVectorIterators::Iterator<number, constness>
internal::BlockVectorIterators::Iterator<number, constness>::operator-(const
int&) const
include/lac/block_vector.h:1756:23: error: 'difference_type' does not name a
type
include/lac/block_vector.h:1756:23: note: (perhaps 'typename
std::iterator<std::random_access_iterator_tag, typename
internal::BlockVectorIterators::Types<number,
constness>::NumberType>::difference_type' was intended)
include/lac/block_vector.h:1756:40: error: ISO C++ forbids declaration of 'd'
with no type [-fpermissive]
include/lac/block_vector.h:1781:23: error: 'difference_type' does not name a
type
include/lac/block_vector.h:1781:23: note: (perhaps 'typename
std::iterator<std::random_access_iterator_tag, typename
internal::BlockVectorIterators::Types<number,
constness>::NumberType>::difference_type' was intended)
include/lac/block_vector.h:1781:40: error: ISO C++ forbids declaration of 'd'
with no type [-fpermissive]
include/lac/block_vector.h:1806:24: error: 'difference_type' does not name a
type
include/lac/block_vector.h:1806:24: note: (perhaps 'typename
std::iterator<std::random_access_iterator_tag, typename
internal::BlockVectorIterators::Types<number,
constness>::NumberType>::difference_type' was intended)
include/lac/block_vector.h:1806:41: error: ISO C++ forbids declaration of 'd'
with no type [-fpermissive]
include/lac/block_vector.h:1834:24: error: 'difference_type' does not name a
type
include/lac/block_vector.h:1834:24: note: (perhaps 'typename
std::iterator<std::random_access_iterator_tag, typename
internal::BlockVectorIterators::Types<number,
constness>::NumberType>::difference_type' was intended)
include/lac/block_vector.h:1834:41: error: ISO C++ forbids declaration of 'd'
with no type [-fpermissive]
specmake[3]: *** [block_vector.o] Error 1
specmake[3]: *** Waiting for unfinished jobs....

Revision 160217 is OK.


-- 
           Summary: [4.6 regression] Failed to build 447.dealII in SPEC CPU
                    2006
           Product: gcc
           Version: 4.6.0
            Status: UNCONFIRMED
          Severity: normal
          Priority: P3
         Component: middle-end
        AssignedTo: unassigned at gcc dot gnu dot org
        ReportedBy: hjl dot tools at gmail dot com


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

[Bug middle-end/44416] [4.6 regression] Failed to build 447.dealII in SPEC CPU 2006

[hjl dot tools at gmail dot com]

------- Comment #1 from hjl dot tools at gmail dot com  2010-06-04 13:26 -------
It may be caused by revision 160231:

http://gcc.gnu.org/ml/gcc-cvs/2010-06/msg00144.html


-- 

hjl dot tools at gmail dot com changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
                 CC|                            |paolo dot carlini at oracle
                   |                            |dot com


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

[Bug middle-end/44416] [4.6 regression] Failed to build 447.dealII in SPEC CPU 2006

[paolo dot carlini at oracle dot com]

------- Comment #2 from paolo dot carlini at oracle dot com  2010-06-04 13:42 -------
I do not have access to those benchmarks. Whoever has, should check that
ptrdiff_t is used properly, either after including <cstddef>, then it can be
used both qualified with std:: and unqualified together with v3, or qualified
with std:: after essentially after including any other c++ header. For sure the
c++ standard doesn't warrant more than the above.


-- 


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

[Bug middle-end/44416] [4.6 regression] Failed to build 447.dealII in SPEC CPU 2006

[singler at kit dot edu]

------- Comment #3 from singler at kit dot edu  2010-06-04 14:19 -------
Bug 44417 is very likely to have the same cause, but here, we can reproduce it
more easily, using the testsuite.

*** This bug has been marked as a duplicate of 44417 ***


-- 

singler at kit dot edu changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
             Status|UNCONFIRMED                 |RESOLVED
         Resolution|                            |DUPLICATE


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

[Bug middle-end/44416] [4.6 regression] Failed to build 447.dealII in SPEC CPU 2006

[paolo dot carlini at oracle dot com]

------- Comment #4 from paolo dot carlini at oracle dot com  2010-06-04 14:29 -------
Not a duplicate, another issue.


-- 

paolo dot carlini at oracle dot com changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
             Status|RESOLVED                    |UNCONFIRMED
         Resolution|DUPLICATE                   |


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

[Bug middle-end/44416] [4.6 regression] Failed to build 447.dealII in SPEC CPU 2006

[jwakely dot gcc at gmail dot com]

------- Comment #5 from jwakely dot gcc at gmail dot com  2010-06-04 14:56 -------
I don't have the SPEC benchmarks either, but probably deal.II uses ptrdiff_t
unqualified, without explicitly including either <cstddef> or <stddef.h>

http://www.dealii.org/developer/doxygen/deal.II/block__vector_8h.html


-- 


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

[Bug middle-end/44416] [4.6 regression] Failed to build 447.dealII in SPEC CPU 2006

[jwakely dot gcc at gmail dot com]

------- Comment #6 from jwakely dot gcc at gmail dot com  2010-06-04 15:00 -------
certainly true for an older version
http://ganymed.iwr.uni-heidelberg.de/~deal/5.2.0/doxygen/deal.II/block__vector_8h.html


-- 


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

[Bug middle-end/44416] [4.6 regression] Failed to build 447.dealII in SPEC CPU 2006

[bangerth at gmail dot com]

------- Comment #7 from bangerth at gmail dot com  2010-06-04 16:36 -------

As the author of the benchmark I can confirm that we apparently forgot
to include the proper header file. So you can call it a defect in 447.dealII.
The question is how to deal with this, of course.

W.


-- 

bangerth at gmail dot com changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
                 CC|                            |bangerth at gmail dot com


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

[Bug middle-end/44416] [4.6 regression] Failed to build 447.dealII in SPEC CPU 2006

[hjl dot tools at gmail dot com]

------- Comment #8 from hjl dot tools at gmail dot com  2010-06-04 16:40 -------
(In reply to comment #7)
> As the author of the benchmark I can confirm that we apparently forgot
> to include the proper header file. So you can call it a defect in 447.dealII.
> The question is how to deal with this, of course.
> 

I will prepare an alt src.


-- 


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

[Bug middle-end/44416] [4.6 regression] Failed to build 447.dealII in SPEC CPU 2006

[hjl dot tools at gmail dot com]

------- Comment #9 from hjl dot tools at gmail dot com  2010-06-04 20:15 -------
Created an attachment (id=20848)
 --> (http://gcc.gnu.org/bugzilla/attachment.cgi?id=20848&action=view)
The src.alt for 447.dealII

This works for me. Can someone try it?


-- 


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

[Bug libstdc++/44416] [4.6 regression] Failed to build 447.dealII in SPEC CPU 2006

[rguenth at gcc dot gnu dot org]

------- Comment #10 from rguenth at gcc dot gnu dot org  2010-07-23 11:58 -------
Invalid.  (alt source works)


-- 

rguenth at gcc dot gnu dot org changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
             Status|UNCONFIRMED                 |RESOLVED
          Component|middle-end                  |libstdc++
         Resolution|                            |INVALID


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

[Bug libstdc++/44416] [4.6 regression] Failed to build 447.dealII in SPEC CPU 2006

[qneill at gcc dot gnu.org]

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

Quentin Neill <qneill at gcc dot gnu.org> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
                 CC|                            |qneill at gcc dot gnu.org

--- Comment #11 from Quentin Neill <qneill at gcc dot gnu.org> 2011-09-08 18:32:38 UTC ---
The apache_stdcxx_4_2_1 srcalt approved for CPU2006 v1.1
(http://www.spec.org/cpu2006/src.alt/README.447.dealII.src.alt.apache_stdcxx_4_2_1.txt)
fixes this problem as well.