================ @@ -93,7 +93,7 @@ class CachedFileSystemEntry { getDirectiveTokens() const { assert(!isError() && "error"); assert(!isDirectory() && "not a file"); - assert(Contents && "contents not initialized"); + // Since isError() and isDirectory() imply that Contents is nullptr, the last assertion is unnecessary ---------------- ayushpareek2003 wrote:
Got it, sir. i will ensure that my comments focus on the code itself rather than describing updates. I’ll keep them clear and concise in my future pull requests https://github.com/llvm/llvm-project/pull/130667 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits