[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