[llvm-testresults] buildbot failure in lab.llvm.org on phase2 - living

[llvmlab-buildmaster at lab.llvm.org]

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

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

Buildslave for this Build: macpro1

Build Reason: scheduler
Build Source Stamp: 204915
Blamelist: alexfh,chandlerc,chapuni,dsanders,erikjv,klimek

BUILD FAILED: failed

sincerely,
 -The Buildbot


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

CHANGES:
File: include/llvm/Support/Allocator.h
On: smooshlab-project
At: Thu 27 Mar 2014 03:01:45
Changed By: chandlerc
Comments: [cleanup] Modernize doxygen comments for the BumpPtrAllocator and
rewrite some of them to be more clear.

The terminology being used in our allocators is making me really sad. We
call things slab allocators that aren't at all slab allocators. It is
quite confusing.Properties: 
  phase_id: r204908-t20140327_030912-b18467



File: include/llvm/Support/Allocator.h
On: smooshlab-project
At: Thu 27 Mar 2014 03:05:45
Changed By: chandlerc
Comments: [cleanup] Run clang-format over these routines to remove formatting
differences from subsequent diffs, and ease review. Going to be
performing some major surgery to simplify this stuff.Properties: 
  phase_id: r204908-t20140327_030912-b18467



Files:
 clang-tidy/ClangTidy.cpp
 test/clang-tidy/fix.cpp
On: smooshlab-project
At: Thu 27 Mar 2014 03:35:45
Changed By: alexfh
Comments: Output messages when clang-tidy applies fix-its.

Reviewers: djasper, klimek

Reviewed By: djasper

CC: cfe-commits

Differential Revision: http://llvm-reviews.chandlerc.com/D3192Properties: 
  phase_id: r204909-t20140327_034159-b18468



Files:
 include/clang/Edit/EditedSource.h
 tools/libclang/IndexingContext.h
On: smooshlab-project
At: Thu 27 Mar 2014 03:55:45
Changed By: chandlerc
Comments: [cleanup] Stop specifying size overrides for BumpPtrAllocators.

These don't seem to have any real point. Let's start with
IndexingContext. I can't come up with any conceivable reason to have
many hundereds of thousands of these alive in an address space which
would make the 4x difference in allocated (but unused) memory for the
string scratch buffer a significant memory usage problem.

The EditedSource one is somewhat more surprising. This is an 8x increase
in the memory allocated (but not used) per editted source file. However,
for this to realistically be a problem, you would need to have over half
a million editted source files in a single address space, and even that
would only really have problems on 32-bit Windows where you really only
have 2gb of virtual address space. And what's more important, the fix to
this if it is actually an issue shouldn't be to shrink the allocator's
size, it is to pass a single allocator into *many* edited source file
objects and let them share the memory.

These were the only two uses of custom sized BumpPtrAllocators
(excluding ones in the JIT using a custom allocation strategy) in all of
LLVM, Clang, LLD, LLDB, or Polly. I don't think we actually need this
complexity in the primary BumpPtrAllocator at all and am planning to
remove it.Properties: 
  phase_id: r204911-t20140327_035746-b18469



Files:
 lib/Target/Mips/MipsISelLowering.cpp
 lib/Target/Mips/MipsISelLowering.h
 lib/Target/Mips/MipsSEISelLowering.cpp
On: smooshlab-project
At: Thu 27 Mar 2014 03:55:45
Changed By: dsanders
Comments: [mips] Stop caching the result of hasMips64(), isABI_O32(), isABI_N32(), and isABI_N64() from MipsSubTarget in MipsTargetLowering

Summary:
The short name is quite convenient so provide an accessor for them instead.

No functional change

Depends on D3177

Reviewers: matheusalmeida

Reviewed By: matheusalmeida

Differential Revision: http://llvm-reviews.chandlerc.com/D3178Properties: 
  phase_id: r204911-t20140327_035746-b18469



Files:
 lib/Target/README.txt
 lib/Transforms/InstCombine/InstCombineCompares.cpp
 test/Transforms/InstCombine/icmp.ll
On: smooshlab-project
At: Thu 27 Mar 2014 04:25:47
Changed By: erikjv
Comments: InstCombine: merge constants in both operands of icmp.

Transform:
    icmp X+Cst2, Cst
into:
    icmp X, Cst-Cst2
when Cst-Cst2 does not overflow, and the add has nsw.Properties: 
  phase_id: r204913-t20140327_042748-b18470



Files:
 lib/Format/TokenAnnotator.cpp
 unittests/Format/FormatTest.cpp
On: smooshlab-project
At: Thu 27 Mar 2014 04:25:47
Changed By: klimek
Comments: Fix bool expression special case.

Clang-format now correctly formats:
  some_type<a * b> v;
  template <bool a, bool b> typename enabled_if<a && b>::type f() {}Properties: 
  phase_id: r204913-t20140327_042748-b18470



File: unittests/IR/InstructionsTest.cpp
On: smooshlab-project
At: Thu 27 Mar 2014 04:41:47
Changed By: chapuni
Comments: IRTests/InstructionsTest.cpp: Avoid initializer list.Properties: 
  phase_id: r204915-t20140327_044348-b18471



File: unittests/IR/InstructionsTest.cpp
On: smooshlab-project
At: Thu 27 Mar 2014 04:41:47
Changed By: chapuni
Comments: SmallVector<3> may be used here.Properties: 
  phase_id: r204915-t20140327_044348-b18471



LOGS: