[PATCH] D92480: [llvm] Unify interface of (ThreadSafe)?RefCountedBase

Nathan James via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Wed Dec 2 08:21:55 PST 2020


njames93 added inline comments.


================
Comment at: llvm/include/llvm/ADT/IntrusiveRefCntPtr.h:120-121
+  ~ThreadSafeRefCountedBase() {
+    assert(RefCount == 0 &&
+           "Destruction occured when there are still references to this.");
+  }
----------------
Hmmm... This assert is causing clang-tidy tests to fail??? Investigating.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D92480



More information about the llvm-commits mailing list