public inbox for gcc-bugs@sourceware.org
help / color / mirror / Atom feed
* [Bug c++/48289] New: [4.5/4.6/4.7 regression] -pedantic breaks std::move
@ 2011-03-25 15:42 jason at gcc dot gnu.org
2011-03-25 15:42 ` [Bug c++/48289] " jason at gcc dot gnu.org
` (4 more replies)
0 siblings, 5 replies; 6+ messages in thread
From: jason at gcc dot gnu.org @ 2011-03-25 15:42 UTC (permalink / raw)
To: gcc-bugs
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=48289
Summary: [4.5/4.6/4.7 regression] -pedantic breaks std::move
Product: gcc
Version: 4.6.0
Status: UNCONFIRMED
Keywords: rejects-valid
Severity: normal
Priority: P3
Component: c++
AssignedTo: jason@gcc.gnu.org
ReportedBy: jason@gcc.gnu.org
>From https://bugzilla.redhat.com/show_bug.cgi?id=690548
Description of problem:
[18:03:50 pal@underdark ~/tmp/1]$ cat a.cpp
#include <utility>
class A { };
static void g ( A && ) { }
template < class T > class B {
public:
void f ( ) {
A a;
g ( std :: move ( a ) );
}
};
=================
[18:04:13 pal@underdark ~/tmp/1]$ LANG=C g++ -std=c++0x -pedantic -c a.cpp
a.cpp: In member function 'void B<T>::f()':
a.cpp:11:25: error: invalid initialization of reference of type 'A&&' from
expression of type 'std::remove_reference<A&>::type'
a.cpp:5:13: error: in passing argument 1 of 'void g(A&&)'
=================
error can be fixed by removing -pedantic or by commenting out /*template <
class T > */
^ permalink raw reply [flat|nested] 6+ messages in thread
* [Bug c++/48289] [4.5/4.6/4.7 regression] -pedantic breaks std::move
2011-03-25 15:42 [Bug c++/48289] New: [4.5/4.6/4.7 regression] -pedantic breaks std::move jason at gcc dot gnu.org
@ 2011-03-25 15:42 ` jason at gcc dot gnu.org
2011-03-25 17:04 ` jason at gcc dot gnu.org
` (3 subsequent siblings)
4 siblings, 0 replies; 6+ messages in thread
From: jason at gcc dot gnu.org @ 2011-03-25 15:42 UTC (permalink / raw)
To: gcc-bugs
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=48289
Jason Merrill <jason at gcc dot gnu.org> changed:
What |Removed |Added
----------------------------------------------------------------------------
Status|UNCONFIRMED |NEW
Last reconfirmed| |2011.03.25 15:20:36
Known to work| |4.4.6
Target Milestone|--- |4.5.3
Ever Confirmed|0 |1
Known to fail| |4.5.3, 4.6.0, 4.7.0
^ permalink raw reply [flat|nested] 6+ messages in thread
* [Bug c++/48289] [4.5/4.6/4.7 regression] -pedantic breaks std::move
2011-03-25 15:42 [Bug c++/48289] New: [4.5/4.6/4.7 regression] -pedantic breaks std::move jason at gcc dot gnu.org
2011-03-25 15:42 ` [Bug c++/48289] " jason at gcc dot gnu.org
@ 2011-03-25 17:04 ` jason at gcc dot gnu.org
2011-03-25 17:54 ` jason at gcc dot gnu.org
` (2 subsequent siblings)
4 siblings, 0 replies; 6+ messages in thread
From: jason at gcc dot gnu.org @ 2011-03-25 17:04 UTC (permalink / raw)
To: gcc-bugs
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=48289
--- Comment #1 from Jason Merrill <jason at gcc dot gnu.org> 2011-03-25 16:17:00 UTC ---
Author: jason
Date: Fri Mar 25 16:16:53 2011
New Revision: 171461
URL: http://gcc.gnu.org/viewcvs?root=gcc&view=rev&rev=171461
Log:
PR c++/48289
* pt.c (build_non_dependent_expr): Keep dereferences outside the
NON_DEPENDENT_EXPR.
Added:
trunk/gcc/testsuite/g++.dg/cpp0x/move1.C
Modified:
trunk/gcc/cp/ChangeLog
trunk/gcc/cp/pt.c
trunk/gcc/testsuite/ChangeLog
^ permalink raw reply [flat|nested] 6+ messages in thread
* [Bug c++/48289] [4.5/4.6/4.7 regression] -pedantic breaks std::move
2011-03-25 15:42 [Bug c++/48289] New: [4.5/4.6/4.7 regression] -pedantic breaks std::move jason at gcc dot gnu.org
2011-03-25 15:42 ` [Bug c++/48289] " jason at gcc dot gnu.org
2011-03-25 17:04 ` jason at gcc dot gnu.org
@ 2011-03-25 17:54 ` jason at gcc dot gnu.org
2011-03-28 15:58 ` jason at gcc dot gnu.org
2011-03-29 14:38 ` jason at gcc dot gnu.org
4 siblings, 0 replies; 6+ messages in thread
From: jason at gcc dot gnu.org @ 2011-03-25 17:54 UTC (permalink / raw)
To: gcc-bugs
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=48289
--- Comment #2 from Jason Merrill <jason at gcc dot gnu.org> 2011-03-25 16:17:41 UTC ---
Author: jason
Date: Fri Mar 25 16:17:36 2011
New Revision: 171463
URL: http://gcc.gnu.org/viewcvs?root=gcc&view=rev&rev=171463
Log:
PR c++/48289
* pt.c (build_non_dependent_expr): Keep dereferences outside the
NON_DEPENDENT_EXPR.
Added:
branches/gcc-4_5-branch/gcc/testsuite/g++.dg/cpp0x/move1.C
Modified:
branches/gcc-4_5-branch/gcc/cp/ChangeLog
branches/gcc-4_5-branch/gcc/cp/pt.c
branches/gcc-4_5-branch/gcc/testsuite/ChangeLog
^ permalink raw reply [flat|nested] 6+ messages in thread
* [Bug c++/48289] [4.5/4.6/4.7 regression] -pedantic breaks std::move
2011-03-25 15:42 [Bug c++/48289] New: [4.5/4.6/4.7 regression] -pedantic breaks std::move jason at gcc dot gnu.org
` (2 preceding siblings ...)
2011-03-25 17:54 ` jason at gcc dot gnu.org
@ 2011-03-28 15:58 ` jason at gcc dot gnu.org
2011-03-29 14:38 ` jason at gcc dot gnu.org
4 siblings, 0 replies; 6+ messages in thread
From: jason at gcc dot gnu.org @ 2011-03-28 15:58 UTC (permalink / raw)
To: gcc-bugs
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=48289
--- Comment #3 from Jason Merrill <jason at gcc dot gnu.org> 2011-03-28 15:06:32 UTC ---
Author: jason
Date: Mon Mar 28 15:06:28 2011
New Revision: 171607
URL: http://gcc.gnu.org/viewcvs?root=gcc&view=rev&rev=171607
Log:
Revert:
PR c++/48289
* pt.c (build_non_dependent_expr): Keep dereferences outside the
NON_DEPENDENT_EXPR.
Removed:
branches/gcc-4_5-branch/gcc/testsuite/g++.dg/cpp0x/move1.C
Modified:
branches/gcc-4_5-branch/gcc/cp/ChangeLog
branches/gcc-4_5-branch/gcc/cp/pt.c
branches/gcc-4_5-branch/gcc/testsuite/ChangeLog
^ permalink raw reply [flat|nested] 6+ messages in thread
* [Bug c++/48289] [4.5/4.6/4.7 regression] -pedantic breaks std::move
2011-03-25 15:42 [Bug c++/48289] New: [4.5/4.6/4.7 regression] -pedantic breaks std::move jason at gcc dot gnu.org
` (3 preceding siblings ...)
2011-03-28 15:58 ` jason at gcc dot gnu.org
@ 2011-03-29 14:38 ` jason at gcc dot gnu.org
4 siblings, 0 replies; 6+ messages in thread
From: jason at gcc dot gnu.org @ 2011-03-29 14:38 UTC (permalink / raw)
To: gcc-bugs
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=48289
Jason Merrill <jason at gcc dot gnu.org> changed:
What |Removed |Added
----------------------------------------------------------------------------
Status|NEW |RESOLVED
Resolution| |FIXED
Target Milestone|4.5.3 |4.6.1
--- Comment #4 from Jason Merrill <jason at gcc dot gnu.org> 2011-03-29 14:35:50 UTC ---
Fixed for 4.6.1. This patch alone wasn't enough to fix the bug in 4.5, so I'm
not going to try to fix it there.
^ permalink raw reply [flat|nested] 6+ messages in thread
end of thread, other threads:[~2011-03-29 14:36 UTC | newest]
Thread overview: 6+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2011-03-25 15:42 [Bug c++/48289] New: [4.5/4.6/4.7 regression] -pedantic breaks std::move jason at gcc dot gnu.org
2011-03-25 15:42 ` [Bug c++/48289] " jason at gcc dot gnu.org
2011-03-25 17:04 ` jason at gcc dot gnu.org
2011-03-25 17:54 ` jason at gcc dot gnu.org
2011-03-28 15:58 ` jason at gcc dot gnu.org
2011-03-29 14:38 ` jason at gcc dot gnu.org
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for read-only IMAP folder(s) and NNTP newsgroup(s).