From: Daniel Dunbar <daniel@zuster.org>
Date: Sat, 10 Apr 2010 01:17:16 +0000 (+0000)
Subject: Disable diag::err_file_modified on Win32 completely, until someone cares to fix
X-Git-Url: https://granicus.if.org/sourcecode?a=commitdiff_plain;h=0b3c773bdc4f5d5f3aaf882b2f7a5f14df6041e0;p=clang

Disable diag::err_file_modified on Win32 completely, until someone cares to fix
it. PR6812.
 - This is another attempt at silencing annoying buildbot failures.

git-svn-id: https://llvm.org/svn/llvm-project/cfe/trunk@100914 91177308-0d34-0410-b5e6-96231b3b80d8
---

diff --git a/lib/Basic/SourceManager.cpp b/lib/Basic/SourceManager.cpp
index 27a859c9d7..053cfe333d 100644
--- a/lib/Basic/SourceManager.cpp
+++ b/lib/Basic/SourceManager.cpp
@@ -98,24 +98,26 @@ const llvm::MemoryBuffer *ContentCache::getBuffer(Diagnostic &Diag,
           << Entry->getName() << ErrorStr;
 
       Buffer.setInt(true);
-    } else if (FileInfo.st_size != Entry->getSize()
+
+    // FIXME: This conditionalization is horrible, but we see spurious failures
+    // in the test suite due to this warning and no one has had time to hunt it
+    // down. So for now, we just don't emit this diagnostic on Win32, and hope
+    // nothing bad happens.
+    //
+    // PR6812.
 #if !defined(LLVM_ON_WIN32)
-               // In our regression testing, the Windows file system
-               // seems to have inconsistent modification times that
-               // sometimes erroneously trigger this error-handling
-               // path.
-               || FileInfo.st_mtime != Entry->getModificationTime()
-#endif
-               ) {
+    } else if (FileInfo.st_size != Entry->getSize() ||
+               FileInfo.st_mtime != Entry->getModificationTime()) {
       // Check that the file's size and modification time are the same
       // as in the file entry (which may have come from a stat cache).
       if (Diag.isDiagnosticInFlight())
-        Diag.SetDelayedDiagnostic(diag::err_file_modified, 
+        Diag.SetDelayedDiagnostic(diag::err_file_modified,
                                   Entry->getName());
-      else 
+      else
         Diag.Report(diag::err_file_modified) << Entry->getName();
 
       Buffer.setInt(true);
+#endif
     }
   }