[Bug libstdc++/66078] New: 20_util/specialized_algorithms/uninitialized_copy/808590.cc fails with -std=c++11

[Bug libstdc++/66078] New: 20_util/specialized_algorithms/uninitialized_copy/808590.cc fails with -std=c++11

[jason at gcc dot gnu.org]

https://gcc.gnu.org/bugzilla/show_bug.cgi?id=66078

            Bug ID: 66078
           Summary: 20_util/specialized_algorithms/uninitialized_copy/8085
                    90.cc fails with -std=c++11
           Product: gcc
           Version: 6.0
            Status: UNCONFIRMED
          Keywords: wrong-code
          Severity: normal
          Priority: P3
         Component: libstdc++
          Assignee: unassigned at gcc dot gnu.org
          Reporter: jason at gcc dot gnu.org
                CC: jwakely.gcc at gmail dot com
  Target Milestone: ---

When compiled with -std=c++11, vector::_M_default_append ends up calling the
template constructor for moving from the old array to the new one.  This seems
wrong as well as inconsistent with C++98; I would expect that the construction
should be from T&& or const T&, not T&.

This failed as far back as 4.7; I have not tested with earlier libraries.

[Bug libstdc++/66078] 20_util/specialized_algorithms/uninitialized_copy/808590.cc fails with -std=c++11

[redi at gcc dot gnu.org]

https://gcc.gnu.org/bugzilla/show_bug.cgi?id=66078

Jonathan Wakely <redi at gcc dot gnu.org> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
             Status|UNCONFIRMED                 |ASSIGNED
   Last reconfirmed|                            |2015-05-08
           Assignee|unassigned at gcc dot gnu.org      |redi at gcc dot gnu.org
     Ever confirmed|0                           |1

--- Comment #1 from Jonathan Wakely <redi at gcc dot gnu.org> ---
We've discussed this on the libstdc++ list and I have a fix ready.

[Bug libstdc++/66078] 20_util/specialized_algorithms/uninitialized_copy/808590.cc fails with -std=c++11

[jason at gcc dot gnu.org]

https://gcc.gnu.org/bugzilla/show_bug.cgi?id=66078

--- Comment #2 from Jason Merrill <jason at gcc dot gnu.org> ---
Ping?  I think this is now the only thing preventing me from throwing the
switch to default C++14.

[Bug libstdc++/66078] 20_util/specialized_algorithms/uninitialized_copy/808590.cc fails with -std=c++11

[redi at gcc dot gnu.org]

https://gcc.gnu.org/bugzilla/show_bug.cgi?id=66078

--- Comment #3 from Jonathan Wakely <redi at gcc dot gnu.org> ---
Author: redi
Date: Wed May 20 17:11:03 2015
New Revision: 223449

URL: https://gcc.gnu.org/viewcvs?rev=223449&root=gcc&view=rev
Log:
        PR libstdc++/66078
        * include/bits/stl_iterator.h (__make_move_if_noexcept_iterator): Add
        overload for pointers.
        * testsuite/20_util/specialized_algorithms/uninitialized_copy/
        808590.cc: Add -std=gnu++03 switch.
        * testsuite/20_util/specialized_algorithms/uninitialized_copy/
        808590-cxx11.cc: Copy of 808590.cc to test with -std=gnu++11.
        * testsuite/23_containers/vector/modifiers/push_back/
        strong_guarantee.cc: New.

Added:
   
trunk/libstdc++-v3/testsuite/20_util/specialized_algorithms/uninitialized_copy/808590-cxx11.cc
      - copied, changed from r223442,
trunk/libstdc++-v3/testsuite/20_util/specialized_algorithms/uninitialized_copy/808590.cc
   
trunk/libstdc++-v3/testsuite/23_containers/vector/modifiers/push_back/strong_guarantee.cc
Modified:
    trunk/libstdc++-v3/ChangeLog
    trunk/libstdc++-v3/include/bits/stl_iterator.h
   
trunk/libstdc++-v3/testsuite/20_util/specialized_algorithms/uninitialized_copy/808590.cc

[Bug libstdc++/66078] 20_util/specialized_algorithms/uninitialized_copy/808590.cc fails with -std=c++11

[redi at gcc dot gnu.org]

https://gcc.gnu.org/bugzilla/show_bug.cgi?id=66078

Jonathan Wakely <redi at gcc dot gnu.org> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
             Status|ASSIGNED                    |RESOLVED
         Resolution|---                         |FIXED
   Target Milestone|---                         |6.0

--- Comment #4 from Jonathan Wakely <redi at gcc dot gnu.org> ---
Fixed on trunk.