[llvm-testresults] buildbot failure in smooshlab on llvm-gcc-powerpc-darwin9

[daniel_dunbar at apple.com]

The Buildbot has detected a new failure of llvm-gcc-powerpc-darwin9 on smooshlab.
Full details are available at:
 http://smooshlab.apple.com:8010/builders/llvm-gcc-powerpc-darwin9/builds/3294

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

Buildslave for this Build: smoosh-14.apple.com

Build Reason: 
Build Source Stamp: 107290
Blamelist: baldrick,cornedbee,fjahanian

BUILD FAILED: failed failed slave lost

sincerely,
 -The Buildbot


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

CHANGES:
Files:
 include/llvm/ADT/SmallPtrSet.h
 lib/Support/SmallPtrSet.cpp
At: Wed 30 Jun 2010 08:04:02
Changed By: baldrick
Comments: Rather than giving SmallPtrSetImpl a member field SmallArray which is magically
replaced by a bigger array in SmallPtrSet (by overridding it), instead just use a
pointer to the start of the storage, and have SmallPtrSet pass in the value to use.
This has the disadvantage that SmallPtrSet becomes bigger by one pointer.  It has
the advantage that it no longer uses tricky C++ rules, and is clearly correct while
I'm not sure the previous version was.  This was inspired by g++-4.6 pointing out
that SmallPtrSetImpl was writing off the end of SmallArray, which it was.  Since
SmallArray is replaced with a bigger array in SmallPtrSet, the write was still to
valid memory.  But it was writing off the end of the declared array type - sounds
kind of dubious to me, like it sounded dubious to g++-4.6.  Maybe g++-4.6 is wrong
and this construct is perfectly valid and correctly compiled by all compilers, but
I think it is better to avoid the whole can of worms by avoiding this construct.
Properties: 




File: include/llvm/ADT/SmallPtrSet.h
At: Wed 30 Jun 2010 08:33:54
Changed By: baldrick
Comments: Clarify that the NextPowerOfTwo template is idempotent.
Properties: 




Files:
 clang.xcodeproj/project.pbxproj
 lib/AST/StmtPrinter.cpp
 test/Rewriter/rewrite-elaborated-type.mm
At: Wed 30 Jun 2010 09:33:54
Changed By: fjahanian
Comments: Nasty rewriter bug which turns out to have an easy fix in
rewriting a c-style cast expression in statement printer.
Fixes radar 8143056.
Properties: 




File: lib/Sema/SemaInit.cpp
At: Wed 30 Jun 2010 09:43:55
Changed By: cornedbee
Comments: Fix a comment typo.Properties: 




LOGS:
Last 10 lines of 'stdio':
	checking for setjmp.h... yes
	checking signal.h usability... yes
	checking signal.h presence... yes
	checking for signal.h... yes
	checking for stdint.h... (cached) yes
	checking termios.h usability... yes
	checking termios.h presence... yes
	checking for termios.h... yes
	checking for unistd.h... (cached) yes
	checking utime.h usability... 

Last 10 lines of 'interrupt':
	[Failure instance: Traceback (failure with no frames): <class 'twisted.internet.error.ConnectionLost'>: Connection to the other side was lost in a non-clean fashion.
	]