mirror of https://github.com/llvm-mirror/libcxx
You cannot select more than 25 topics
Topics must start with a letter or number, can include dashes ('-') and can be up to 35 characters long.
598ae4fa2e
------------------------------------------------------------------------
r315994 | ericwf | 2017-10-17 06:03:17 -0700 (Tue, 17 Oct 2017) | 18 lines
[libc++] Fix PR34898 - vector iterator constructors and assign method perform push_back instead of emplace_back.
Summary:
The constructors `vector(Iter, Iter, Alloc = Alloc{})` and `assign(Iter, Iter)` don't correctly perform EmplaceConstruction from the result of dereferencing the iterator. This results in them performing an additional and unneeded copy.
This patch addresses the issue by correctly using `emplace_back` in C++11 and newer.
There are also some bugs in our `insert` implementation, but those will be handled separately.
@mclow.lists We should probably merge this into 5.1, agreed?
Reviewers: mclow.lists, dlj, EricWF
Reviewed By: mclow.lists, EricWF
Subscribers: cfe-commits, mclow.lists
Differential Revision: https://reviews.llvm.org/D38757
------------------------------------------------------------------------
git-svn-id: https://llvm.org/svn/llvm-project/libcxx/branches/release_50@318837 91177308-0d34-0410-b5e6-96231b3b80d8
|
8 years ago | |
|---|---|---|
| benchmarks | 9 years ago | |
| cmake | 8 years ago | |
| docs | 8 years ago | |
| include | 8 years ago | |
| lib | 8 years ago | |
| src | 8 years ago | |
| test | 8 years ago | |
| utils | 8 years ago | |
| www | 8 years ago | |
| .arcconfig | 9 years ago | |
| .clang-format | 9 years ago | |
| .gitignore | 9 years ago | |
| CMakeLists.txt | 8 years ago | |
| CREDITS.TXT | 9 years ago | |
| LICENSE.TXT | 9 years ago | |
| NOTES.TXT | 9 years ago | |
| TODO.TXT | 9 years ago | |
| appveyor-reqs-install.cmd | 9 years ago | |
| appveyor.yml | 9 years ago | |
docs/README.txt
libc++ Documentation ==================== The libc++ documentation is written using the Sphinx documentation generator. It is currently tested with Sphinx 1.1.3. To build the documents into html configure libc++ with the following cmake options: * -DLLVM_ENABLE_SPHINX=ON * -DLIBCXX_INCLUDE_DOCS=ON After configuring libc++ with these options the make rule `docs-libcxx-html` should be available.