================
@@ -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

Reply via email to