[PATCH] D30837: [libcxx] Support for shared_ptr<T()>

Eric Fiselier via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Fri Mar 10 15:08:00 PST 2017


EricWF requested changes to this revision.
EricWF added a comment.
This revision now requires changes to proceed.

We can't just use an arbitrary allocator type for a number of reasons:

- You just changed the type of the control block. That's ABI breaking.
- `allocator<int>` allocates ints, nothing else.
- It may mean we don't select a valid user specialization of `allocator<Yp>`.

I'll play around to see if I can come up with another path forward to fix this.


https://reviews.llvm.org/D30837





More information about the cfe-commits mailing list