[PATCH] D150525: [libc++][memory] P1132R8: out_ptr - a scalable output pointer abstraction

Hristo Hristov via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Sat Oct 21 00:55:21 PDT 2023


H-G-Hristov added inline comments.


================
Comment at: libcxx/include/__memory/inout_ptr.h:34
+  static_assert(!__is_specialization_v<_Smart, shared_ptr>, "std::shared_ptr<> is not supported");
+
+public:
----------------
philnik wrote:
> Maybe also `static_assert` Cpp17NullablePointer?
I made an attempt to implement this requirement but I don't think it was anywhere near correct, so I removed it from this revision. I'd be glad for some advice with this.


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D150525/new/

https://reviews.llvm.org/D150525



More information about the llvm-commits mailing list