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

[daniel_dunbar at apple.com]

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/11782

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

Buildslave for this Build: smoosh-04

Build Reason: 
Build Source Stamp: 119341
Blamelist: chandlerc,sfider

BUILD FAILED: failed compile

sincerely,
 -The Buildbot


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

CHANGES:
Files:
 include/clang/Basic/DiagnosticSemaKinds.td
 lib/Sema/SemaDeclAttr.cpp
 test/SemaCXX/attr-format.cpp
 test/SemaCXX/attr-nonnull.cpp
 test/SemaCXX/format-attribute.cpp
At: Tue 16 Nov 2010 00:42:37
Changed By: chandlerc
Comments: Re-work the handling of implicit 'this' arguments and silly GCC-style attribute
argument indexes. This handles the offsets in a consistent manner for all of
the attributes which I saw working with these concepts. I've also added tests
for the attribute that motivated this: nonnull.

I consolidated the tests for format attributes into one file, and fleshed them
out a bit to trigger more of the warning cases. Also improved the quality of
some of the diagnostics that occur with invalid argument indices.

The only really questionable change here is supporting the implicit this
argument for the ownership attribute. I'm not sure it's really a sensible
concept there, but implemented the logic for consistency.
Properties: 




Files:
 lib/Sema/SemaChecking.cpp
 test/SemaCXX/attr-format.cpp
At: Tue 16 Nov 2010 00:52:38
Changed By: chandlerc
Comments: Fix PR8625 and correctly interpret member-calls to static members when
producing warnings.

This feels really fragile, and I've not audited all other argument index-based
warnings. I suspect we'll grow this bug on another warning eventually. It might
be nice to adjust the argument indices when building up the attribute AST node,
as we already have to remember about the 'this' argument within that code to
produce correct errors.
Properties: 




File: lib/Checker/GRCXXExprEngine.cpp
At: Tue 16 Nov 2010 00:52:38
Changed By: sfider
Comments: Refactored GRExprEngine::getCXXThisRegion to use CXXMethodDecl::getThisType instead of calculating it by hand.
Properties: 




LOGS:
Last 10 lines of 'stdio':
	llvm[4]: Compiling TextDiagnosticBuffer.cpp for Debug+Asserts build
	llvm[4]: Compiling TextDiagnosticPrinter.cpp for Debug+Asserts build
	llvm[4]: Compiling TypeXML.cpp for Debug+Asserts build
	llvm[4]: Compiling VerifyDiagnosticsClient.cpp for Debug+Asserts build
	llvm[4]: Building Debug+Asserts Archive Library libclangIndex.a
	llvm[4]: Compiling Warnings.cpp for Debug+Asserts build
	llvm[4]: Building Debug+Asserts Archive Library libclangFrontend.a
	make[2]: *** [all] Error 1
	make[1]: *** [clang/.makeall] Error 2
	make: *** [all] Error 1