[PATCH] D47109: LWG 2969 "polymorphic_allocator::construct() shouldn't pass resource()"

Arthur O'Dwyer via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Sat May 19 12:16:46 PDT 2018


Quuxplusone created this revision.
Quuxplusone added a reviewer: EricWF.
Herald added a subscriber: cfe-commits.

In the TS, `uses_allocator` construction for `pair` tried to use an allocator type of `memory_resource*`, which is incorrect because `memory_resource*` is not an allocator type. LWG 2969 fixed it to use `polymorphic_allocator` as the allocator type instead.
https://wg21.link/lwg2969

(https://reviews.llvm.org/D47090 included this in `<memory_resource>`; at Eric's request, I've split this out into its own patch applied to the existing `<experimental/memory_resource>` instead.)


Repository:
  rCXX libc++

https://reviews.llvm.org/D47109

Files:
  include/experimental/memory_resource
  test/std/experimental/memory/memory.polymorphic.allocator.class/memory.polymorphic.allocator.mem/construct_pair_const_lvalue_pair.pass.cpp
  test/std/experimental/memory/memory.polymorphic.allocator.class/memory.polymorphic.allocator.mem/construct_pair_rvalue.pass.cpp
  test/std/experimental/memory/memory.polymorphic.allocator.class/memory.polymorphic.allocator.mem/construct_pair_values.pass.cpp
  test/std/experimental/memory/memory.polymorphic.allocator.class/memory.polymorphic.allocator.mem/construct_piecewise_pair.pass.cpp
  test/std/experimental/memory/memory.polymorphic.allocator.class/memory.polymorphic.allocator.mem/construct_piecewise_pair_evil.pass.cpp
  test/std/experimental/memory/memory.polymorphic.allocator.class/memory.polymorphic.allocator.mem/construct_types.pass.cpp
  test/support/test_memory_resource.hpp

-------------- next part --------------
A non-text attachment was scrubbed...
Name: D47109.147674.patch
Type: text/x-patch
Size: 20800 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/cfe-commits/attachments/20180519/8f7813b9/attachment-0001.bin>


More information about the cfe-commits mailing list