[cfe-commits] r169830 - /cfe/trunk/lib/Serialization/ASTReader.cpp
Galina Kistanova
gkistanova at gmail.com
Mon Dec 17 16:31:52 PST 2012
Hi Argyrios,
r169831 broke the test, for 169830 all tests pass:
http://lab.llvm.org:8011/builders/clang-X86_64-freebsd/builds/6149.
Thanks
Galina
On Thu, Dec 13, 2012 at 4:48 PM, Argyrios Kyrtzidis <akyrtzi at gmail.com> wrote:
> On Dec 13, 2012, at 4:02 PM, Galina Kistanova <gkistanova at gmail.com> wrote:
>
>> Hi Argyrios,
>>
>> One of these commits (169830, 169831) added failing test to
>> clang-X86_64-freebsd builder:
>> Clang :: Misc/dev-fd-fs.c failed
>>
>> The builder is here:
>> http://lab.llvm.org:8011/builders/clang-X86_64-freebsd
>>
>> For r 169829 all tests pass:
>> http://lab.llvm.org:8011/builders/clang-X86_64-freebsd/builds/6142
>>
>> Please have a look at this?
>
> r169830 is unrelated and I don't see what the issue would be with r169831.
>
> r169831 changed the way we stat, in that if a file descriptor is not needed, then we call 'stat' directly instead of opening the file and then calling fstat on it.
> Does this not work with named pipes on freebsd ?
>
>>
>> Thanks
>>
>> Galina
>>
>>
>>
>> On Mon, Dec 10, 2012 at 11:48 PM, Argyrios Kyrtzidis <akyrtzi at gmail.com> wrote:
>>> Author: akirtzidis
>>> Date: Tue Dec 11 01:48:14 2012
>>> New Revision: 169830
>>>
>>> URL: http://llvm.org/viewvc/llvm-project?rev=169830&view=rev
>>> Log:
>>> [PCH] Remove redundant stat call when validating the input files.
>>>
>>> This became redundant after we removed the stat cache.
>>>
>>> Modified:
>>> cfe/trunk/lib/Serialization/ASTReader.cpp
>>>
>>> Modified: cfe/trunk/lib/Serialization/ASTReader.cpp
>>> URL: http://llvm.org/viewvc/llvm-project/cfe/trunk/lib/Serialization/ASTReader.cpp?rev=169830&r1=169829&r2=169830&view=diff
>>> ==============================================================================
>>> --- cfe/trunk/lib/Serialization/ASTReader.cpp (original)
>>> +++ cfe/trunk/lib/Serialization/ASTReader.cpp Tue Dec 11 01:48:14 2012
>>> @@ -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 @@
>>> 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)
>>>
>>>
>>> _______________________________________________
>>> cfe-commits mailing list
>>> cfe-commits at cs.uiuc.edu
>>> http://lists.cs.uiuc.edu/mailman/listinfo/cfe-commits
>
More information about the cfe-commits
mailing list