From: Argyrios Kyrtzidis <akyrtzi@gmail.com>
Date: Tue, 11 Dec 2012 07:48:14 +0000 (+0000)
Subject: [PCH] Remove redundant stat call when validating the input files.
X-Git-Url: http://review.tizen.org/git/?a=commitdiff_plain;h=53c3877f94635ed77ccb35cd1dffc3b44e42dff4;p=platform%2Fupstream%2Fllvm.git

[PCH] Remove redundant stat call when validating the input files.

This became redundant after we removed the stat cache.

llvm-svn: 169830
---

diff --git a/clang/lib/Serialization/ASTReader.cpp b/clang/lib/Serialization/ASTReader.cpp
index e6d7951..5c25f58 100644
--- a/clang/lib/Serialization/ASTReader.cpp
+++ b/clang/lib/Serialization/ASTReader.cpp
@@ -53,7 +53,6 @@
 #include <algorithm>
 #include <cstdio>
 #include <iterator>
-#include <sys/stat.h>
 
 using namespace clang;
 using namespace clang::serialization;
@@ -1560,20 +1559,12 @@ ASTReader::getInputFile(ModuleFile &F, unsigned ID, bool Complain) {
     if (Overridden)
       return InputFile(File, Overridden);
 
-    // The stat info from the FileEntry came from the cached stat
-    // info of the PCH, so we cannot trust it.
-    struct stat StatBuf;
-    if (::stat(File->getName(), &StatBuf) != 0) {
-      StatBuf.st_size = File->getSize();
-      StatBuf.st_mtime = File->getModificationTime();
-    }
-
-    if ((StoredSize != StatBuf.st_size
+    if ((StoredSize != File->getSize()
 #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.
-         || StoredTime != StatBuf.st_mtime
+         || StoredTime != File->getModificationTime()
 #endif
          )) {
       if (Complain)