[llvm-testresults] buildbot failure in smooshlab on clang-x86_64-darwin10

daniel_dunbar at apple.com daniel_dunbar at apple.com
Mon Nov 22 23:06:30 PST 2010


The Buildbot has detected a new failure of clang-x86_64-darwin10 on smooshlab.
Full details are available at:
 http://smooshlab.apple.com:8010/builders/clang-x86_64-darwin10/builds/11984

Buildbot URL: http://smooshlab.apple.com:8010/

Buildslave for this Build: smoosh-04

Build Reason: 
Build Source Stamp: 120001
Blamelist: fpichet,lattner

BUILD FAILED: failed test-clang

sincerely,
 -The Buildbot


================================================================================

CHANGES:
Files:
 include/llvm/Support/MemoryBuffer.h
 lib/Support/MemoryBuffer.cpp
At: Mon 22 Nov 2010 22:14:26
Changed By: lattner
Comments: Revert functionality doug added in r98575 that was never
documented and only used by some clang stuff I just removed.
Properties: 




Files:
 include/clang/Basic/DiagnosticSemaKinds.td
 include/clang/Sema/Sema.h
 lib/Sema/SemaDecl.cpp
 test/CodeGen/ms-anonymous-struct.c
 test/Sema/MicrosoftExtensions.c
At: Mon 22 Nov 2010 22:14:26
Changed By: fpichet
Comments: Microsoft C anonymous struct implementation.
Documentation: http://msdn.microsoft.com/en-us/library/z2cx9y4f.aspxProperties: 




Files:
 include/clang/Basic/FileManager.h
 include/clang/Frontend/ASTUnit.h
 lib/Basic/FileManager.cpp
 lib/Basic/SourceManager.cpp
 lib/Frontend/ASTUnit.cpp
At: Mon 22 Nov 2010 22:14:26
Changed By: lattner
Comments: Partially revert Doug's PCH validation patch (r98585).

This patch completely defeated the "passing in a prestat'd size 
to MemoryBuffer" optimization, leading to an extra fstat call for
every buffer opened, in order to find out if the datestamp and size 
of the file on disk matches what is in the stat cache.

I fully admit that I don't completely understand what is going on here:
why punish code when a stat cache isn't in use?  what is the point of a
stat cache if you have to turn around and stat stuff to validate it?

To resolve both these issues, just drop the modtime check and check the
file size, which is the important thing anyway.  This should also resolve
PR6812, because presumably windows is stable when it comes to file sizes.
If the modtime is actually important, we should get it and keep it on the
first stat.

This eliminates 833 fstat syscalls when processing Cocoa.h, speeding up
system time on -Eonly Cocoa.h  from 0.041 to 0.038s.

Properties: 




LOGS:
Last 10 lines of 'stdio':
	Testing Time: 108.29s
	********************
	Failing Tests (1):
	    Clang :: CodeGen/ms-anonymous-struct.c
	
	  Expected Passes    : 2662
	  Expected Failures  : 20
	  Unexpected Failures: 1
	make[1]: *** [all] Error 1
	make: *** [test] Error 2

Last 10 lines of 'warnings':
	/Users/buildslave/zorg/buildbot/smooshlab/slave/build.clang-x86_64-darwin10/llvm/tools/clang/test/CodeGen/ms-anonymous-struct.c(43) :  warning: expression result unused
	/Users/buildslave/zorg/buildbot/smooshlab/slave/build.clang-x86_64-darwin10/llvm/tools/clang/test/CodeGen/ms-anonymous-struct.c(49) :  warning: expression result unused
	/Users/buildslave/zorg/buildbot/smooshlab/slave/build.clang-x86_64-darwin10/llvm/tools/clang/test/CodeGen/ms-anonymous-struct.c(53) :  warning: expression result unused
	/Users/buildslave/zorg/buildbot/smooshlab/slave/build.clang-x86_64-darwin10/llvm/tools/clang/test/CodeGen/ms-anonymous-struct.c(57) :  warning: expression result unused
	/Users/buildslave/zorg/buildbot/smooshlab/slave/build.clang-x86_64-darwin10/llvm/tools/clang/test/CodeGen/ms-anonymous-struct.c(68) :  warning: expression result unused
	/Users/buildslave/zorg/buildbot/smooshlab/slave/build.clang-x86_64-darwin10/llvm/tools/clang/test/CodeGen/ms-anonymous-struct.c(74) :  warning: expression result unused
	/Users/buildslave/zorg/buildbot/smooshlab/slave/build.clang-x86_64-darwin10/llvm/tools/clang/test/CodeGen/ms-anonymous-struct.c(81) :  warning: expression result unused
	/Users/buildslave/zorg/buildbot/smooshlab/slave/build.clang-x86_64-darwin10/llvm/tools/clang/test/CodeGen/ms-anonymous-struct.c(88) :  warning: expression result unused
	/Users/buildslave/zorg/buildbot/smooshlab/slave/build.clang-x86_64-darwin10/llvm/tools/clang/test/CodeGen/ms-anonymous-struct.c(93) :  warning: expression result unused
	/Users/buildslave/zorg/buildbot/smooshlab/slave/build.clang-x86_64-darwin10/llvm/tools/clang/test/CodeGen/ms-anonymous-struct.c(98) :  warning: expression result unused

Last 10 lines of 'fail':
	Clang :: CodeGen/ms-anonymous-struct.c

Last 10 lines of 'xfail':
	Clang :: FixIt/fixit-errors.c
	Clang :: FixIt/fixit-pmem.cpp
	Clang :: FixIt/typo.m
	Clang :: Lexer/hexfloat.cpp
	Clang :: PCH/changed-files.c
	Clang :: PCH/pr4489.c
	Clang :: PCH/source-manager-stack.c
	Clang :: Parser/cxx-ambig-decl-expr-xfail.cpp
	Clang :: SemaCXX/rval-references-xfail.cpp
	Clang :: SemaTemplate/instantiate-function-1.mm

Last 10 lines of 'ms-anonymous-struct.c':
	<stdin>:1:1: note: scanning from here
	; ModuleID = '/Users/buildslave/zorg/buildbot/smooshlab/slave/build.clang-x86_64-darwin10/llvm/tools/clang/test/CodeGen/ms-anonymous-struct.c'
	^
	<stdin>:5:1: note: possible intended match here
	%struct.nested1 = type { i64, i64 }
	^
	--
	
	********************
	




More information about the llvm-testresults mailing list