[Bug libstdc++/52822] [C++11] stable_partition destroys sequence due to inappropriate self-move-assignment

2012-04-12 Thread paolo.carlini at oracle dot com
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=52822 --- Comment #8 from Paolo Carlini paolo.carlini at oracle dot com 2012-04-12 12:21:04 UTC --- Jeffrey, please commit and close the PR. Thanks.

[Bug libstdc++/52822] [C++11] stable_partition destroys sequence due to inappropriate self-move-assignment

2012-04-12 Thread jyasskin at gcc dot gnu.org
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=52822 --- Comment #9 from Jeffrey Yasskin jyasskin at gcc dot gnu.org 2012-04-12 20:59:14 UTC --- Author: jyasskin Date: Thu Apr 12 20:59:09 2012 New Revision: 186391 URL: http://gcc.gnu.org/viewcvs?root=gccview=revrev=186391 Log: Fix PR52822

[Bug libstdc++/52822] [C++11] stable_partition destroys sequence due to inappropriate self-move-assignment

2012-04-12 Thread jyasskin at gcc dot gnu.org
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=52822 --- Comment #10 from Jeffrey Yasskin jyasskin at gcc dot gnu.org 2012-04-12 21:42:00 UTC --- Author: jyasskin Date: Thu Apr 12 21:41:55 2012 New Revision: 186394 URL: http://gcc.gnu.org/viewcvs?root=gccview=revrev=186394 Log: Fix PR52822

[Bug libstdc++/52822] [C++11] stable_partition destroys sequence due to inappropriate self-move-assignment

2012-04-12 Thread jyasskin at gcc dot gnu.org
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=52822 --- Comment #11 from Jeffrey Yasskin jyasskin at gcc dot gnu.org 2012-04-12 22:26:08 UTC --- Author: jyasskin Date: Thu Apr 12 22:26:02 2012 New Revision: 186396 URL: http://gcc.gnu.org/viewcvs?root=gccview=revrev=186396 Log: Fix PR52822 by

[Bug libstdc++/52822] [C++11] stable_partition destroys sequence due to inappropriate self-move-assignment

2012-04-12 Thread jyasskin at gcc dot gnu.org
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=52822 Jeffrey Yasskin jyasskin at gcc dot gnu.org changed: What|Removed |Added Status|NEW |RESOLVED

[Bug libstdc++/52822] [C++11] stable_partition destroys sequence due to inappropriate self-move-assignment

2012-04-02 Thread paolo.carlini at oracle dot com
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=52822 Paolo Carlini paolo.carlini at oracle dot com changed: What|Removed |Added CC|

[Bug libstdc++/52822] [C++11] stable_partition destroys sequence due to inappropriate self-move-assignment

2012-04-02 Thread jyasskin at gcc dot gnu.org
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=52822 --- Comment #7 from Jeffrey Yasskin jyasskin at gcc dot gnu.org 2012-04-02 16:41:41 UTC --- Sounds good. Will send the patches to the list, probably tomorrow. Thanks!

[Bug libstdc++/52822] [C++11] stable_partition destroys sequence due to inappropriate self-move-assignment

2012-04-01 Thread jyasskin at gcc dot gnu.org
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=52822 --- Comment #1 from Jeffrey Yasskin jyasskin at gcc dot gnu.org 2012-04-01 20:41:43 UTC --- Running the 4.6 testsuite with -std=c++0x as the default was unsuccessful, in that it found only one error in an execution test, which was already known,

[Bug libstdc++/52822] [C++11] stable_partition destroys sequence due to inappropriate self-move-assignment

2012-04-01 Thread redi at gcc dot gnu.org
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=52822 Jonathan Wakely redi at gcc dot gnu.org changed: What|Removed |Added Status|UNCONFIRMED |NEW Last

[Bug libstdc++/52822] [C++11] stable_partition destroys sequence due to inappropriate self-move-assignment

2012-04-01 Thread paolo.carlini at oracle dot com
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=52822 --- Comment #3 from Paolo Carlini paolo.carlini at oracle dot com 2012-04-01 21:50:42 UTC --- Lately, in 4.6 too, in the occasion of a similar issue, we also added a check for self-move assignment to __gnu_test::rvalstruct. Evidently, the small

[Bug libstdc++/52822] [C++11] stable_partition destroys sequence due to inappropriate self-move-assignment

2012-04-01 Thread paolo.carlini at oracle dot com
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=52822 --- Comment #4 from Paolo Carlini paolo.carlini at oracle dot com 2012-04-01 22:09:45 UTC --- PS: then shall we just do the assignment only when __result1 != __first, or algorithmically we can do better? In case we can also imagine having the

[Bug libstdc++/52822] [C++11] stable_partition destroys sequence due to inappropriate self-move-assignment

2012-04-01 Thread jyasskin at gcc dot gnu.org
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=52822 --- Comment #5 from Jeffrey Yasskin jyasskin at gcc dot gnu.org 2012-04-02 02:15:24 UTC --- Created attachment 27058 -- http://gcc.gnu.org/bugzilla/attachment.cgi?id=27058 Fix by skipping past true-predicate values Here's an algorithmic fix