dblaikie added a comment. I've committed 2e83ccc2ee333389110659f3cb313968a0c970d4 <https://reviews.llvm.org/rG2e83ccc2ee333389110659f3cb313968a0c970d4> which takes parts of this patch - specifically fixing the test that was already intended to test the RefCountedBase's copy constructor, but didn't (because the test called the default constructor instead) and extending it to test/cover (& also fix) the similar behavior in ThreadSafeRefCountedBase.
That should leave this proposed patch to add the dtors and assertions. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D92480/new/ https://reviews.llvm.org/D92480 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
