[libcxx-commits] [PATCH] D93130: [libc++] Refactor allocate_shared to use an allocation guard
Louis Dionne via Phabricator via libcxx-commits
libcxx-commits at lists.llvm.org
Tue Dec 15 08:33:28 PST 2020
ldionne added a comment.
In D93130#2453963 <https://reviews.llvm.org/D93130#2453963>, @saugustine wrote:
> This change appears to have dropped a constructor, or maybe two. I'm not familiar enough with all the requirements to know if that is correct or not. Here is a somewhat redacted version of the errors I get building a private application.
Is it possible that your custom `Allocator` has its converting copy constructor (from other specializations of `Allocator`) marked as `explicit`? We don't seem to have a test for that, I'll add one and fix this, but please confirm that it's the case.
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
More information about the libcxx-commits