[llvm-testresults] buildbot failure in lab.llvm.org on phase3 - tree health

Tue Nov 12 03:31:10 PST 2013

The Buildbot has detected a new failure on builder phase3 - tree health while building lab.llvm.org.
Full details are available at:
 http://lab.llvm.org:8013/builders/phase3%20-%20tree%20health/builds/2151

Buildbot URL: http://lab.llvm.org:8013/

Buildslave for this Build: macpro1

Build Reason: scheduler
Build Source Stamp: 194449
Blamelist: aadg,arsenm,faisalv,jtsoftware,rkotler,rnk,rsmith

BUILD FAILED: failed

sincerely,
 -The Buildbot

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

CHANGES:
File: lib/CodeGen/CalcSpillWeights.cpp
On: smooshlab-project
At: Fri 08 Nov 2013 07:20:30
Changed By: aadg
Comments: CalculateSpillWeights cleanup: remove unneeded includesProperties: 
  phase_id: r194259-t20131108_072231-b13873

File: lib/Bitcode/Reader/BitcodeReader.cpp
On: smooshlab-project
At: Mon 11 Nov 2013 17:36:13
Changed By: rnk
Comments: Extract a bc attr parsing helper that returns Attribute::None on error

The parsing method still returns llvm::error_code for consistency with
other parsing methods.  Minor cleanup, no functionality change.Properties: 
  phase_id: r194439-t20131111_180421-b13875

Files:
 include/clang/Sema/Sema.h
 lib/Sema/Sema.cpp
 lib/Sema/SemaDecl.cpp
 lib/Sema/TreeTransform.h
On: smooshlab-project
At: Mon 11 Nov 2013 17:46:13
Changed By: faisalv
Comments: REFACTOR: Have PushLambdaScope return the LambdaScopeInfo that it creates.

No Functionality change.

This refactoring avoids having to call getCurLambda right after PushLambdaScope, to obtain the LambdaScopeInfo that was created during the call to PushLambdaScope. Properties: 
  phase_id: r194439-t20131111_180421-b13875

File: lib/Sema/Sema.cpp
On: smooshlab-project
At: Mon 11 Nov 2013 17:56:13
Changed By: faisalv
Comments: COSMETIC: Right justify an asterix in the previous refactoring.

Hopefully Richard won't notice this terrible egregiocity - clearly the work of a malevolent poltergeist - fixed now ;)

No functionality change.Properties: 
  phase_id: r194439-t20131111_180421-b13875

File: test/pp-trace/pp-trace-conditional.cpp
On: smooshlab-project
At: Mon 11 Nov 2013 18:10:13
Changed By: jtsoftware
Comments: [extra] pp-trace - Add test for conditional callbacks.Properties: 
  phase_id: r194443-t20131111_184719-b13879

Files:
 lib/AST/Expr.cpp
 test/SemaCXX/MicrosoftCompatibility.cpp
On: smooshlab-project
At: Mon 11 Nov 2013 18:30:13
Changed By: rnk
Comments: -fms-compatibility: Use C++98 null pointer constant rules

Patch by Will Wilson!Properties: 
  phase_id: r194443-t20131111_184719-b13879

Files:
 lib/Target/Mips/Mips16InstrInfo.cpp
 lib/Target/Mips/Mips16InstrInfo.td
 lib/Target/Mips/MipsConstantIslandPass.cpp
 test/CodeGen/Mips/simplebr.ll
On: smooshlab-project
At: Mon 11 Nov 2013 18:36:13
Changed By: rkotler
Comments: Change the default branch instruction to be the 16 bit variety for mips16.
This has no material effect at this time since we don't have a direct
object emitter for mips16 and the assembler can't tell them apart. I
place a comment "16 bit inst" for those so that I can tell them apart in the
output. The constant island pass has only been minimally changed to allow
this. More complete branch work is forthcoming but this is the first
step.

Properties: 
  phase_id: r194443-t20131111_184719-b13879

Files:
 lib/Target/R600/InstPrinter/AMDGPUInstPrinter.cpp
 lib/Target/R600/InstPrinter/AMDGPUInstPrinter.h
 test/CodeGen/R600/32-bit-local-address-space.ll
 test/CodeGen/R600/64bit-kernel-args.ll
 test/CodeGen/R600/add.ll
 test/CodeGen/R600/address-space.ll
 test/CodeGen/R600/and.ll
 test/CodeGen/R600/bfi_int.ll
 test/CodeGen/R600/build_vector.ll
 test/CodeGen/R600/fabs.ll
 test/CodeGen/R600/fadd64.ll
 test/CodeGen/R600/fcmp64.ll
 test/CodeGen/R600/fconst64.ll
 test/CodeGen/R600/fdiv64.ll
 test/CodeGen/R600/fma.ll
 test/CodeGen/R600/fmul64.ll
 test/CodeGen/R600/fmuladd.ll
 test/CodeGen/R600/fneg.ll
 test/CodeGen/R600/fsqrt.ll
 test/CodeGen/R600/fsub64.ll
 test/CodeGen/R600/imm.ll
 test/CodeGen/R600/indirect-addressing-si.ll
 test/CodeGen/R600/kernel-args.ll
 test/CodeGen/R600/llvm.SI.imageload.ll
 test/CodeGen/R600/llvm.SI.resinfo.ll
 test/CodeGen/R600/llvm.SI.sample-masked.ll
 test/CodeGen/R600/llvm.SI.sample.ll
 test/CodeGen/R600/llvm.SI.sampled.ll
 test/CodeGen/R600/llvm.SI.tbuffer.store.ll
 test/CodeGen/R600/load.ll
 test/CodeGen/R600/load.vec.ll
 test/CodeGen/R600/load64.ll
 test/CodeGen/R600/local-memory-two-objects.ll
 test/CodeGen/R600/local-memory.ll
 test/CodeGen/R600/lshl.ll
 test/CodeGen/R600/lshr.ll
 test/CodeGen/R600/mad_uint24.ll
 test/CodeGen/R600/mul.ll
 test/CodeGen/R600/mul_uint24.ll
 test/CodeGen/R600/mulhu.ll
 test/CodeGen/R600/or.ll
 test/CodeGen/R600/rotr.ll
 test/CodeGen/R600/seto.ll
 test/CodeGen/R600/setuo.ll
 test/CodeGen/R600/sgpr-copy.ll
 test/CodeGen/R600/shl.ll
 test/CodeGen/R600/si-lod-bias.ll
 test/CodeGen/R600/sra.ll
 test/CodeGen/R600/srl.ll
 test/CodeGen/R600/sub.ll
 test/CodeGen/R600/trunc.ll
 test/CodeGen/R600/unaligned-load-store.ll
 test/CodeGen/R600/work-item-intrinsics.ll
 test/CodeGen/R600/xor.ll
 test/CodeGen/R600/zero_extend.ll
On: smooshlab-project
At: Mon 11 Nov 2013 18:40:13
Changed By: arsenm
Comments: R600/SI: Change formatting of printed registers.

Print the range of registers used with a single letter prefix.
This better matches what the shader compiler produces and
is overall less obnoxious than concatenating all of the
subregister names together.

Instead of SGPR0, it will print s0. Instead of SGPR0_SGPR1,
it will print s[0:1] and so on.

There doesn't appear to be a straightforward way
to get the actual register info in the InstPrinter,
so this parses the generated name to print with the
new syntax.

The required test changes are pretty nasty, and register
matching regexes are now worse. Since there isn't a way to
add to a variable in FileCheck, some of the tests now don't
check the exact number of registers used, but I don't think that
will be a real problem.Properties: 
  phase_id: r194443-t20131111_184719-b13879

Files:
 include/clang/AST/ASTDiagnostic.h
 include/clang/AST/CommentDiagnostic.h
 include/clang/Analysis/AnalysisDiagnostic.h
 include/clang/Basic/Diagnostic.td
 include/clang/Basic/DiagnosticIDs.h
 include/clang/Basic/DiagnosticSemaKinds.td
 include/clang/Driver/DriverDiagnostic.h
 include/clang/Frontend/FrontendDiagnostic.h
 include/clang/Lex/LexDiagnostic.h
 include/clang/Parse/ParseDiagnostic.h
 include/clang/Sema/SemaDiagnostic.h
 include/clang/Serialization/SerializationDiagnostic.h
 lib/Basic/DiagnosticIDs.cpp
 lib/Sema/SemaInit.cpp
 lib/Sema/SemaOverload.cpp
 tools/diagtool/DiagnosticNames.cpp
 utils/TableGen/ClangDiagnosticsEmitter.cpp
On: smooshlab-project
At: Mon 11 Nov 2013 18:50:13
Changed By: rsmith
Comments: Rather than duplicating extension diagnostics to allow them to cause a
substitution failure, allow a flag to be set on the Diagnostic object,
to mark it as 'causes substitution failure'.

Refactor Diagnostic.td and the tablegen to use an enum for SFINAE behavior
rather than a bunch of flags.
Properties: 
  phase_id: r194447-t20131111_193226-b13882

File: test/SemaTemplate/extension-sfinae.cpp
On: smooshlab-project
At: Mon 11 Nov 2013 18:50:13
Changed By: rsmith
Comments: Extra test for r194444.
Properties: 
  phase_id: r194447-t20131111_193226-b13882

File: lib/Basic/DiagnosticIDs.cpp
On: smooshlab-project
At: Mon 11 Nov 2013 19:06:13
Changed By: rsmith
Comments: Remove C++11ism.
Properties: 
  phase_id: r194447-t20131111_193226-b13882

Files:
 pp-trace/PPCallbacksTracker.cpp
 test/pp-trace/pp-trace-ident.cpp
 test/pp-trace/pp-trace-modules.cpp
On: smooshlab-project
At: Mon 11 Nov 2013 19:20:13
Changed By: jtsoftware
Comments: Fixed moduleImport and ident callbacks.  Fixed modules test. Added ident callback test.  A better test for moduleExport is coming.Properties: 
  phase_id: r194447-t20131111_193226-b13882

Files:
 lib/Sema/SemaExprCXX.cpp
 test/SemaCXX/cxx1y-generic-lambdas.cpp
On: smooshlab-project
At: Mon 11 Nov 2013 19:56:13
Changed By: faisalv
Comments: A quick fix to PR17877 that was introduced by r194188 (generic-lambda-capturing) that broke libc++.

See http://lists.cs.uiuc.edu/pipermail/cfe-dev/2013-November/033369.html for discussion on cfe-dev.

This fix explicitly checks whether we are within the declcontext of a lambda's call operator - which is what I had intended to be true (and assumed would be true if getCurLambda returns a valid pointer) before checking whether a lambda can capture the potential-captures of the innermost lambda.

A deeper fix (that addresses why getCurLambda() returns a valid pointer when perhaps it shouldn't?) - as proposed by Richard Smith in http://llvm.org/bugs/show_bug.cgi?id=17877 - has been suggested as a FIXME.

Patch was LGTM'd by Richard (just barely :)

http://llvm-reviews.chandlerc.com/D2144Properties: 
  phase_id: r194449-t20131111_200716-b13884

File: lib/Sema/SemaExprCXX.cpp
On: smooshlab-project
At: Mon 11 Nov 2013 20:00:13
Changed By: faisalv
Comments: COSMETIC: Fix 80 column overflow in some comments introduced in r194188

no functionality change.Properties: 
  phase_id: r194449-t20131111_200716-b13884

LOGS: