dblaikie accepted this revision.
dblaikie added a comment.
This revision is now accepted and ready to land.
Looks good with one minor tweak
================
Comment at: llvm/include/llvm/ADT/IntrusiveRefCntPtr.h:180
+ template <class X>
+ IntrusiveRefCntPtr(std::unique_ptr<X> &&S) : Obj(S.release()) {
+ retain();
----------------
Pass 'S' by value here - that's the usual way to pass ownership of unique_ptr,
and unlike the IntrusiveRefCntPtr's copy/move ctors there's no issue with
invoking the operation in its implementation (infinitely recursively).
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D92888/new/
https://reviews.llvm.org/D92888
_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
