[Bug c++/43140] New: [4.5 Regression] FAIL: g++.dg/graphite/pr41305.C (test for excess errors)

[Bug c++/43140] New: [4.5 Regression] FAIL: g++.dg/graphite/pr41305.C (test for excess errors)

[dominiq at lps dot ens dot fr]

Between revisions 156927 and 156932 g++.dg/graphite/pr41305.C has started to
fail (likely due to revision 156928):

[macbook] f90/bug% gcc45 -c -O3 -floop-interchange
/opt/gcc/work/gcc/testsuite/g++.dg/graphite/pr41305.C
/opt/gcc/work/gcc/testsuite/g++.dg/graphite/pr41305.C: In member function
'pointer allocator<_Tp>::allocate(allocator::size_type) [with _Tp = double,
pointer = double, allocator::size_type = long unsigned int]':
/opt/gcc/work/gcc/testsuite/g++.dg/graphite/pr41305.C:80:5:   instantiated from
'unbounded_array<T, <template-parameter-1-2>
>::unbounded_array(unbounded_array<T, <template-parameter-1-2> >::size_type,
ALLOC) [with T = double, ALLOC = allocator<double>, unbounded_array<T,
<template-parameter-1-2> >::size_type = long unsigned int]'
/opt/gcc/work/gcc/testsuite/g++.dg/graphite/pr41305.C:272:83:   instantiated
from 'vector<T, <template-parameter-1-2> >::vector(vector<T,
<template-parameter-1-2> >::size_type, value_type) [with T = double, A =
unbounded_array<double, allocator<double> >, vector<T, <template-parameter-1-2>
>::size_type = long unsigned int, value_type = double]'
/opt/gcc/work/gcc/testsuite/g++.dg/graphite/pr41305.C:323:32:   instantiated
from here
/opt/gcc/work/gcc/testsuite/g++.dg/graphite/pr41305.C:24:12: warning:
converting to non-pointer type 'allocator<double>::pointer' from NULL


-- 
           Summary: [4.5 Regression] FAIL: g++.dg/graphite/pr41305.C (test
                    for excess errors)
           Product: gcc
           Version: 4.5.0
            Status: UNCONFIRMED
          Severity: normal
          Priority: P3
         Component: c++
        AssignedTo: unassigned at gcc dot gnu dot org
        ReportedBy: dominiq at lps dot ens dot fr


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

[Bug c++/43140] [4.5 Regression] FAIL: g++.dg/graphite/pr41305.C (test for excess errors)

[dominiq at lps dot ens dot fr]

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



------- Comment #1 from dominiq at lps dot ens dot fr  2010-02-22 12:55 -------
CC Manuel López-Ibáñez and H.J. Lu.


-- 

dominiq at lps dot ens dot fr changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
                 CC|                            |hjl dot tools at gmail dot
                   |                            |com, manu at gcc dot gnu dot
                   |                            |org


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

[Bug c++/43140] [4.5 Regression] FAIL: g++.dg/graphite/pr41305.C (test for excess errors)

[manu at gcc dot gnu dot org]

------- Comment #2 from manu at gcc dot gnu dot org  2010-02-22 13:49 -------
It seems to me that the template parameter to allocator should be a pointer and
in this testcase is just double. So the warning is correct. If the testcase is
correct, then you should either catch the warning with // { dg-warning } or
silence it with // { dg-options "-Wno-conversion-null" }


-- 

manu at gcc dot gnu dot org changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
             Status|UNCONFIRMED                 |NEW
     Ever Confirmed|0                           |1
   Last reconfirmed|0000-00-00 00:00:00         |2010-02-22 13:49:05
               date|                            |


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

[Bug c++/43140] [4.5 Regression] FAIL: g++.dg/graphite/pr41305.C (test for excess errors)

[dominiq at lps dot ens dot fr]

------- Comment #3 from dominiq at lps dot ens dot fr  2010-02-22 14:14 -------
CC Sebastian Pop.


-- 

dominiq at lps dot ens dot fr changed:

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


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

[Bug c++/43140] [4.5 Regression] FAIL: g++.dg/graphite/pr41305.C (test for excess errors)

[spop at gcc dot gnu dot org]

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



------- Comment #4 from spop at gcc dot gnu dot org  2010-02-22 14:55 -------
Subject: Bug 43140

Author: spop
Date: Mon Feb 22 14:55:08 2010
New Revision: 156963

URL: http://gcc.gnu.org/viewcvs?root=gcc&view=rev&rev=156963
Log:
Fix PR43140: Add -Wno-conversion-null to pr41305.C.

2010-02-22  Sebastian Pop  <sebastian.pop@amd.com>
            Manuel López-Ibáñez  <manu@gcc.gnu.org>

        PR middle-end/43140
        * g++.dg/graphite/pr41305.C: Add -Wno-conversion-null.

Modified:
    branches/graphite/gcc/ChangeLog.graphite
    branches/graphite/gcc/testsuite/g++.dg/graphite/pr41305.C


-- 


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

[Bug c++/43140] [4.5 Regression] FAIL: g++.dg/graphite/pr41305.C (test for excess errors)

[spop at gcc dot gnu dot org]

------- Comment #5 from spop at gcc dot gnu dot org  2010-02-22 15:13 -------
Subject: Bug 43140

Author: spop
Date: Mon Feb 22 15:13:39 2010
New Revision: 156964

URL: http://gcc.gnu.org/viewcvs?root=gcc&view=rev&rev=156964
Log:
Fix PR43097: rename only SSA_NAMEs.

2010-02-22  Sebastian Pop  <sebastian.pop@amd.com>

        PR middle-end/43140
        * sese.c (get_rename): Assert that old_name is an SSA_NAME.
        (rename_variables_in_stmt): Continue when the use is not an SSA_NAME.

        * gfortran.dg/graphite/pr43097.f: New.

Added:
    branches/graphite/gcc/testsuite/gfortran.dg/graphite/pr43097.f
Modified:
    branches/graphite/gcc/ChangeLog.graphite
    branches/graphite/gcc/sese.c


-- 


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

[Bug c++/43140] [4.5 Regression] FAIL: g++.dg/graphite/pr41305.C (test for excess errors)

[spop at gcc dot gnu dot org]

-- 

spop at gcc dot gnu dot org changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
         AssignedTo|unassigned at gcc dot gnu   |spop at gcc dot gnu dot org
                   |dot org                     |
             Status|NEW                         |ASSIGNED
   Last reconfirmed|2010-02-22 13:49:05         |2010-02-22 15:19:56
               date|                            |


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

[Bug c++/43140] [4.5 Regression] FAIL: g++.dg/graphite/pr41305.C (test for excess errors)

[spop at gcc dot gnu dot org]

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



------- Comment #6 from spop at gcc dot gnu dot org  2010-02-23 12:59 -------
Subject: Bug 43140

Author: spop
Date: Tue Feb 23 12:59:00 2010
New Revision: 156994

URL: http://gcc.gnu.org/viewcvs?root=gcc&view=rev&rev=156994
Log:
Fix PR43140: Add -Wno-conversion-null to pr41305.C.

2010-02-22  Sebastian Pop  <sebastian.pop@amd.com>
            Manuel López-Ibáñez  <manu@gcc.gnu.org>

        PR middle-end/43140
        * g++.dg/graphite/pr41305.C: Add -Wno-conversion-null.

Modified:
    trunk/gcc/ChangeLog.graphite
    trunk/gcc/testsuite/g++.dg/graphite/pr41305.C


-- 


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

[Bug c++/43140] [4.5 Regression] FAIL: g++.dg/graphite/pr41305.C (test for excess errors)

[spop at gcc dot gnu dot org]

------- Comment #7 from spop at gcc dot gnu dot org  2010-02-23 12:59 -------
Subject: Bug 43140

Author: spop
Date: Tue Feb 23 12:59:17 2010
New Revision: 156995

URL: http://gcc.gnu.org/viewcvs?root=gcc&view=rev&rev=156995
Log:
Fix PR43097: rename only SSA_NAMEs.

2010-02-22  Sebastian Pop  <sebastian.pop@amd.com>

        PR middle-end/43140
        * sese.c (get_rename): Assert that old_name is an SSA_NAME.
        (rename_variables_in_stmt): Continue when the use is not an SSA_NAME.

        * gfortran.dg/graphite/pr43097.f: New.

Added:
    trunk/gcc/testsuite/gfortran.dg/graphite/pr43097.f
Modified:
    trunk/gcc/ChangeLog.graphite
    trunk/gcc/sese.c


-- 


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

[Bug c++/43140] [4.5 Regression] FAIL: g++.dg/graphite/pr41305.C (test for excess errors)

[spop at gcc dot gnu dot org]

------- Comment #8 from spop at gcc dot gnu dot org  2010-02-23 13:03 -------
Fixed.


-- 

spop at gcc dot gnu dot org changed:

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


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