[PATCH] D49985: [ADT] ImmutableList no longer requires elements to be copy constructible

Artem Dergachev via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Mon Aug 6 11:46:35 PDT 2018


NoQ added a comment.

But generally, yeah, i guess the main problem with putting non-POD objects into the immutable list is that you cannot easily make the factory call destructors when it dies because the allocator within it is only good in providing chunks of memory, not tracking them.

Putting smart pointers into an immutable list doesn't make it non-POD or harder to copy, but we are still unable to recall what we need to destroy when time comes.


https://reviews.llvm.org/D49985





More information about the llvm-commits mailing list