[PATCH] D94440: [ADT] Add makeIntrusiveRefCnt helper function

Nathan James via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Mon Jan 11 11:49:40 PST 2021


njames93 added inline comments.


================
Comment at: llvm/unittests/ADT/IntrusiveRefCntPtrTest.cpp:62-67
+    static_assert(
+        std::is_same<decltype(S1), IntrusiveRefCntPtr<TypeParam>>::value,
+        "Non-const type mismatch");
+    static_assert(
+        std::is_same<decltype(S2), IntrusiveRefCntPtr<const TypeParam>>::value,
+        "Const type mismatch");
----------------
Couldn't think how best to test this, but these seem to do a good enough job.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D94440



More information about the llvm-commits mailing list