JDevlieghere added inline comments.
================ Comment at: lldb/include/lldb/Core/DataFileCache.h:82 private: + /// Gets the default LLDB index cache policy, which is s ---------------- It's a little weird that `GetLLDBIndexCachePolicy` would be private, if only from a documentation point of view so that you can refer to it in the constructor. ================ Comment at: lldb/include/lldb/Core/DataFileCache.h:83 private: + /// Gets the default LLDB index cache policy, which is s + static llvm::CachePruningPolicy GetLLDBIndexCachePolicy(); ---------------- Comment is incomplete Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D131531/new/ https://reviews.llvm.org/D131531 _______________________________________________ lldb-commits mailing list lldb-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/lldb-commits