nikic wrote: I think this approach is fine, given that the IntrusiveRefCntPtr.h docs explicitly recommend it.
I'm a bit confused by the discussion about Instrumentor.h? That file does not seem to use VFS and is not touched by this PR? https://github.com/llvm/llvm-project/pull/211761 _______________________________________________ cfe-commits mailing list [email protected] https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
