================
@@ -65,8 +65,7 @@ class ExpandModularHeadersPPCallbacks::FileRecorder {
 };
 
 ExpandModularHeadersPPCallbacks::ExpandModularHeadersPPCallbacks(
-    CompilerInstance *CI,
-    IntrusiveRefCntPtr<llvm::vfs::OverlayFileSystem> OverlayFS)
+    CompilerInstance *CI, llvm::vfs::OverlayFileSystem &OverlayFS)
----------------
vbvictor wrote:

How could we know that original object of `OverlayFileSystem &` will outlive 
this reference. From docs I assume that `IntrusiveRefCntPtr` has `shared_ptr` 
semantics but this convertion to reference seems bugprone. 

https://github.com/llvm/llvm-project/pull/163686
_______________________________________________
cfe-commits mailing list
[email protected]
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Reply via email to