[Bug c++/53821] [4.7/4.8 regression][c++11] multiple definitions of lambda

2012-12-06 Thread jason at gcc dot gnu.org


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



--- Comment #10 from Jason Merrill jason at gcc dot gnu.org 2012-12-06 
14:37:25 UTC ---

Author: jason

Date: Thu Dec  6 14:37:13 2012

New Revision: 194251



URL: http://gcc.gnu.org/viewcvs?root=gccview=revrev=194251

Log:

PR c++/55015

PR c++/53821

* semantics.c (maybe_add_lambda_conv_op): Revert earlier change.

* decl.c (start_preparsed_function): Make local class methods comdat

in templates, too.



Added:

trunk/gcc/testsuite/g++.dg/cpp0x/lambda/lambda-conv6.C

Modified:

trunk/gcc/cp/ChangeLog

trunk/gcc/cp/decl.c

trunk/gcc/cp/semantics.c


[Bug c++/53821] [4.7/4.8 regression][c++11] multiple definitions of lambda

2012-12-06 Thread jason at gcc dot gnu.org


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



--- Comment #11 from Jason Merrill jason at gcc dot gnu.org 2012-12-06 
14:39:58 UTC ---

Author: jason

Date: Thu Dec  6 14:39:52 2012

New Revision: 194253



URL: http://gcc.gnu.org/viewcvs?root=gccview=revrev=194253

Log:

PR c++/55015

PR c++/53821

* semantics.c (maybe_add_lambda_conv_op): Revert earlier change.

* decl.c (start_preparsed_function): Make local class methods comdat

in templates, too.



Added:

branches/gcc-4_7-branch/gcc/testsuite/g++.dg/cpp0x/lambda/lambda-conv6.C

Modified:

branches/gcc-4_7-branch/gcc/cp/ChangeLog

branches/gcc-4_7-branch/gcc/cp/decl.c

branches/gcc-4_7-branch/gcc/cp/semantics.c


[Bug c++/53821] [4.7/4.8 regression][c++11] multiple definitions of lambda

2012-07-03 Thread vincenzo.innocente at cern dot ch
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=53821

--- Comment #7 from vincenzo Innocente vincenzo.innocente at cern dot ch 
2012-07-03 20:39:50 UTC ---
I'm still wandering, more in general, if there is a semantic difference between


templatetypename T 
int FooT::bar(int i, int j) {
…
}

and

templatetypename T 
inline
int FooT::bar(int i, int j) {
…
}


[Bug c++/53821] [4.7/4.8 regression][c++11] multiple definitions of lambda

2012-07-03 Thread redi at gcc dot gnu.org
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=53821

Jonathan Wakely redi at gcc dot gnu.org changed:

   What|Removed |Added

 CC||redi at gcc dot gnu.org

--- Comment #8 from Jonathan Wakely redi at gcc dot gnu.org 2012-07-03 
20:55:25 UTC ---
*** Bug 53675 has been marked as a duplicate of this bug. ***


[Bug c++/53821] [4.7/4.8 regression][c++11] multiple definitions of lambda

2012-07-03 Thread jason at gcc dot gnu.org
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=53821

Jason Merrill jason at gcc dot gnu.org changed:

   What|Removed |Added

 CC|jason at redhat dot com |

--- Comment #9 from Jason Merrill jason at gcc dot gnu.org 2012-07-03 
21:28:02 UTC ---
The only difference is the hint to the compiler that you want the function to
be inlined.


[Bug c++/53821] [4.7/4.8 regression][c++11] multiple definitions of lambda

2012-07-02 Thread rguenth at gcc dot gnu.org
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=53821

Richard Guenther rguenth at gcc dot gnu.org changed:

   What|Removed |Added

   Priority|P3  |P1
  Known to work||4.7.1
Summary|[4.7.2,4.8  |[4.7/4.8 regression][c++11]
   |regression][c++11] multiple |multiple definitions of
   |definitions of lambda   |lambda


[Bug c++/53821] [4.7/4.8 regression][c++11] multiple definitions of lambda

2012-07-02 Thread redi at gcc dot gnu.org
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=53821

--- Comment #3 from Jonathan Wakely redi at gcc dot gnu.org 2012-07-02 
10:07:17 UTC ---
This is PR 53675


[Bug c++/53821] [4.7/4.8 regression][c++11] multiple definitions of lambda

2012-07-02 Thread jason at gcc dot gnu.org
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=53821

--- Comment #4 from Jason Merrill jason at gcc dot gnu.org 2012-07-02 
19:14:40 UTC ---
Author: jason
Date: Mon Jul  2 19:14:34 2012
New Revision: 189175

URL: http://gcc.gnu.org/viewcvs?root=gccview=revrev=189175
Log:
PR c++/53821
* semantics.c (maybe_add_lambda_conv_op): Don't set
DECL_INTERFACE_KNOWN.

Added:
trunk/gcc/testsuite/g++.dg/cpp0x/lambda/lambda-template6.C
Modified:
trunk/gcc/cp/ChangeLog
trunk/gcc/cp/semantics.c
trunk/gcc/testsuite/ChangeLog


[Bug c++/53821] [4.7/4.8 regression][c++11] multiple definitions of lambda

2012-07-02 Thread jason at gcc dot gnu.org
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=53821

--- Comment #4 from Jason Merrill jason at gcc dot gnu.org 2012-07-02 
19:14:40 UTC ---
Author: jason
Date: Mon Jul  2 19:14:34 2012
New Revision: 189175

URL: http://gcc.gnu.org/viewcvs?root=gccview=revrev=189175
Log:
PR c++/53821
* semantics.c (maybe_add_lambda_conv_op): Don't set
DECL_INTERFACE_KNOWN.

Added:
trunk/gcc/testsuite/g++.dg/cpp0x/lambda/lambda-template6.C
Modified:
trunk/gcc/cp/ChangeLog
trunk/gcc/cp/semantics.c
trunk/gcc/testsuite/ChangeLog

--- Comment #5 from Jason Merrill jason at gcc dot gnu.org 2012-07-02 
19:15:03 UTC ---
Author: jason
Date: Mon Jul  2 19:14:58 2012
New Revision: 189176

URL: http://gcc.gnu.org/viewcvs?root=gccview=revrev=189176
Log:
PR c++/53821
* semantics.c (maybe_add_lambda_conv_op): Don't set
DECL_INTERFACE_KNOWN.

Added:
   
branches/gcc-4_7-branch/gcc/testsuite/g++.dg/cpp0x/lambda/lambda-template6.C
Modified:
branches/gcc-4_7-branch/gcc/cp/ChangeLog
branches/gcc-4_7-branch/gcc/cp/semantics.c
branches/gcc-4_7-branch/gcc/testsuite/ChangeLog


[Bug c++/53821] [4.7/4.8 regression][c++11] multiple definitions of lambda

2012-07-02 Thread jason at gcc dot gnu.org
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=53821

Jason Merrill jason at gcc dot gnu.org changed:

   What|Removed |Added

 Status|NEW |RESOLVED
 CC||jason at gcc dot gnu.org
 Resolution||FIXED

--- Comment #6 from Jason Merrill jason at gcc dot gnu.org 2012-07-02 
19:25:28 UTC ---
Fixed.