[Bug c++/53565] New: [4.8 Regression] FAIL: libgomp.c++/for-7.C

[Bug c++/53565] New: [4.8 Regression] FAIL: libgomp.c++/for-7.C

[dominiq at lps dot ens.fr]

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

             Bug #: 53565
           Summary: [4.8 Regression] FAIL: libgomp.c++/for-7.C
    Classification: Unclassified
           Product: gcc
           Version: 4.8.0
            Status: UNCONFIRMED
          Severity: normal
          Priority: P3
         Component: c++
        AssignedTo: unassigned@gcc.gnu.org
        ReportedBy: dominiq@lps.ens.fr
                CC: hjl.tools@gmail.com, jason@gcc.gnu.org


Between revisions 188113 (OK, see
http://gcc.gnu.org/ml/gcc-testresults/2012-06/msg00085.html ) and 188118 (see
http://gcc.gnu.org/ml/gcc-testresults/2012-06/msg00094.html ), the test
libgomp.c++/for-7.C has started to fail with:

Executing on host: /opt/gcc/build_w/gcc/xgcc -B/opt/gcc/build_w/gcc/
/opt/gcc/work/libgomp/testsuite/libgomp.c++/for-7.C 
-B/opt/gcc/build_w/x86_64-apple-darwin10.8.0/i386/libgomp/
-B/opt/gcc/build_w/x86_64-apple-darwin10.8.0/i386/libgomp/.libs
-I/opt/gcc/build_w/x86_64-apple-darwin10.8.0/i386/libgomp
-I/opt/gcc/work/libgomp/testsuite/.. -march=i486 -shared-libgcc
-fmessage-length=0 -fno-diagnostics-show-caret -fopenmp  -O0  -std=c++0x
-fopenmp
-B/opt/gcc/build_w/x86_64-apple-darwin10.8.0/i386/libgomp/../libstdc++-v3/src/.libs
  -L/opt/gcc/build_w/x86_64-apple-darwin10.8.0/i386/libgomp/.libs
-L/opt/gcc/build_w/x86_64-apple-darwin10.8.0/i386/libgomp/../libstdc++-v3/src/.libs
-lstdc++ -lm   -m32 -o ./for-7.exe    (timeout = 300)
/opt/gcc/work/libgomp/testsuite/libgomp.c++/for-7.C: In instantiation of 'void
f2() [with T = int]':
/opt/gcc/work/libgomp/testsuite/libgomp.c++/for-7.C:95:12:   required from here
/opt/gcc/work/libgomp/testsuite/libgomp.c++/for-7.C:49:3: error: use of 'i'
before deduction of 'auto'
/opt/gcc/work/libgomp/testsuite/libgomp.c++/for-7.C: In instantiation of 'void
f5() [with T = const char*]':
/opt/gcc/work/libgomp/testsuite/libgomp.c++/for-7.C:104:21:   required from
here
/opt/gcc/work/libgomp/testsuite/libgomp.c++/for-7.C:77:3: error: use of 'i'
before deduction of 'auto'
compiler exited with status 1
...

FAIL: libgomp.c++/for-7.C  -O0  (test for excess errors)
Excess errors:
/opt/gcc/work/libgomp/testsuite/libgomp.c++/for-7.C:49:3: error: use of 'i'
before deduction of 'auto'
/opt/gcc/work/libgomp/testsuite/libgomp.c++/for-7.C:77:3: error: use of 'i'
before deduction of 'auto'

>From the error it seems that this is caused by r188116.

[Bug c++/53565] [4.8 Regression] FAIL: libgomp.c++/for-7.C

[hjl.tools at gmail dot com]

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

H.J. Lu <hjl.tools at gmail dot com> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
             Status|UNCONFIRMED                 |NEW
   Last reconfirmed|                            |2012-06-03
   Target Milestone|---                         |4.8.0
     Ever Confirmed|0                           |1

[Bug c++/53565] [4.8 Regression] FAIL: libgomp.c++/for-7.C

[dominiq at lps dot ens.fr]

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

Dominique d'Humieres <dominiq at lps dot ens.fr> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
                 CC|                            |jakub at redhat dot com

--- Comment #1 from Dominique d'Humieres <dominiq at lps dot ens.fr> 2012-06-15 21:56:49 UTC ---
This is indeed caused/exposed by

http://gcc.gnu.org/viewcvs?view=revision&sortby=date&revision=188116

[Bug c++/53565] [4.8 Regression] FAIL: libgomp.c++/for-7.C

[jason at gcc dot gnu.org]

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

--- Comment #2 from Jason Merrill <jason at gcc dot gnu.org> 2012-06-25 15:17:55 UTC ---
Author: jason
Date: Mon Jun 25 15:17:48 2012
New Revision: 188939

URL: http://gcc.gnu.org/viewcvs?root=gcc&view=rev&rev=188939
Log:
    PR c++/53565
    * pt.c (tsubst_omp_for_iterator): Simplify DECL_EXPR handling.
    (tsubst_expr) [OMP_FOR]: Here, too.

Modified:
    trunk/gcc/cp/ChangeLog
    trunk/gcc/cp/pt.c
    trunk/gcc/testsuite/g++.dg/gomp/for-19.C

[Bug c++/53565] [4.8 Regression] FAIL: libgomp.c++/for-7.C

[jason at gcc dot gnu.org]

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

Jason Merrill <jason at gcc dot gnu.org> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
             Status|NEW                         |RESOLVED
         Resolution|                            |FIXED
         AssignedTo|unassigned at gcc dot       |jason at gcc dot gnu.org
                   |gnu.org                     |

--- Comment #3 from Jason Merrill <jason at gcc dot gnu.org> 2012-06-25 15:46:30 UTC ---
Fixed.