jkorous accepted this revision. jkorous added a comment. This revision is now accepted and ready to land.
In D59388#1433233 <https://reviews.llvm.org/D59388#1433233>, @dexonsmith wrote: > Yes, it's safe. The reference count is "intrusive", meaning it's stored in > the object itself (via inheritance from `RefCountedBase`). As a result, all > the instances of `IntrusiveRefCntPtr` that reference at the same object will > implicitly share their count. I missed that `llvm::vfs::FileSystem` inherits from `ThreadSafeRefCountedBase`. LGTM. CHANGES SINCE LAST ACTION https://reviews.llvm.org/D59388/new/ https://reviews.llvm.org/D59388 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
