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

Rafael EspĂ­ndola rafael.espindola at gmail.com
Sat May 17 14:08:13 PDT 2014


Should be fixed in 209075.

On 17 May 2014 16:11, Rafael EspĂ­ndola <rafael.espindola at gmail.com> wrote:
> I just started a lto bootstrap to see if I can reproduce this.
>
> On 17 May 2014 15:36,  <llvmlab-buildmaster at lab.llvm.org> wrote:
>> 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/2365
>>
>> Buildbot URL: http://lab.llvm.org:8013/
>>
>> Buildslave for this Build: macpro1
>>
>> Build Reason: scheduler
>> Build Source Stamp: 209067
>> Blamelist: akirtzidis,alp,chandlerc,chapuni,compnerd,dblaikie,dexonsmith,filcab,gclayton,jmolenda,louis,majnemer,pete,rafael,rsmith,samsonov
>>
>> BUILD FAILED: failed
>>
>> sincerely,
>>  -The Buildbot
>>
>>
>> ================================================================================
>>
>> CHANGES:
>> File: source/Core/Value.cpp
>> On: smooshlab-project
>> At: Fri 16 May 2014 15:00:20
>> Changed By: gclayton
>> Comments: Fix the copy constructor and assignement operator for the lldb_private::Value class to "do the right thing".
>>
>> Properties:
>>   phase_id: r209045-t20140516_160242-b19995
>>
>>
>>
>> Files:
>>  lib/CodeGen/AsmPrinter/DwarfUnit.cpp
>>  test/DebugInfo/sugared-constants.ll
>> On: smooshlab-project
>> At: Fri 16 May 2014 15:06:20
>> Changed By: dblaikie
>> Comments: DebugInfo: Handle emitting constants of C++ unicode character type.
>>
>> Patch by Stephan Tolksdorf! (with some test case stuff by me)
>>
>> Differential Revision: http://reviews.llvm.org/D3810Properties:
>>   phase_id: r209045-t20140516_160242-b19995
>>
>>
>>
>> File: lib/CodeGen/CodeGenModule.cpp
>> On: smooshlab-project
>> At: Fri 16 May 2014 15:30:20
>> Changed By: rafael
>> Comments: Use getAliasee instead of getAliasedGlobal.
>>
>> No functionality change.Properties:
>>   phase_id: r209045-t20140516_160242-b19995
>>
>>
>>
>> File: lib/CodeGen/AsmPrinter/DwarfDebug.cpp
>> On: smooshlab-project
>> At: Fri 16 May 2014 15:30:20
>> Changed By: dblaikie
>> Comments: DebugInfo: Assert rather than conditionalizing when a CU's subprogram list contains declarations.Properties:
>>   phase_id: r209045-t20140516_160242-b19995
>>
>>
>>
>> Files:
>>  include/llvm/IR/GlobalAlias.h
>>  lib/CodeGen/AsmPrinter/AsmPrinter.cpp
>>  lib/CodeGen/SelectionDAG/SelectionDAG.cpp
>>  lib/ExecutionEngine/JIT/JITEmitter.cpp
>>  lib/IR/Globals.cpp
>>  lib/LTO/LTOModule.cpp
>>  lib/Target/ARM64/ARM64FastISel.cpp
>>  lib/Target/PowerPC/PPCAsmPrinter.cpp
>>  lib/Target/PowerPC/PPCFastISel.cpp
>>  lib/Target/PowerPC/PPCISelDAGToDAG.cpp
>>  lib/Target/TargetMachine.cpp
>>  lib/Target/X86/X86AsmPrinter.cpp
>>  lib/Target/X86/X86FastISel.cpp
>>  lib/Target/X86/X86ISelLowering.cpp
>>  lib/Target/XCore/XCoreISelLowering.cpp
>>  lib/Transforms/Instrumentation/DataFlowSanitizer.cpp
>>  tools/llvm-nm/llvm-nm.cpp
>> On: smooshlab-project
>> At: Fri 16 May 2014 15:46:20
>> Changed By: rafael
>> Comments: Delete getAliasedGlobal.Properties:
>>   phase_id: r209045-t20140516_160242-b19995
>>
>>
>>
>> Files:
>>  lib/Target/X86/X86ISelLowering.cpp
>>  lib/Target/X86/X86ISelLowering.h
>> On: smooshlab-project
>> At: Fri 16 May 2014 16:00:20
>> Changed By: filcab
>> Comments: Implemented LowerVSELECT to custom lower some instructions.
>>
>> No functionality change intended. The types that previously were set to
>> lower as Expand or Legal are doing the same thing with this lowering
>> function.Properties:
>>   phase_id: r209045-t20140516_160242-b19995
>>
>>
>>
>> Files:
>>  lib/Target/X86/X86ISelLowering.cpp
>>  test/CodeGen/X86/avx-blend.ll
>>  test/CodeGen/X86/avx2.ll
>>  test/CodeGen/X86/blend-msb.ll
>>  test/CodeGen/X86/sse41-blend.ll
>>  test/CodeGen/X86/sse41.ll
>> On: smooshlab-project
>> At: Fri 16 May 2014 16:00:20
>> Changed By: filcab
>> Comments: Lower vselects into X86ISD::BLENDI when appropriate.
>>
>> LowerVSELECT will, if possible, generate a X86ISD::BLENDI DAG node if the
>> condition is constant and we can emit that instruction, given the
>> subtarget.
>>
>> This is not enough for all cases. An additional SELECTCombine optimization
>> will be committed.
>>
>> Fixed tests that were expecting variable blends but where a blend+imm can
>> be generated.
>> Added test where we can't emit blend+immediate.
>> Added avx2 blend+imm tests.Properties:
>>   phase_id: r209045-t20140516_160242-b19995
>>
>>
>>
>> Files:
>>  lib/Target/X86/X86ISelLowering.cpp
>>  test/CodeGen/X86/avx-blend.ll
>> On: smooshlab-project
>> At: Fri 16 May 2014 16:00:20
>> Changed By: filcab
>> Comments: Implemented special cases for PerformVSELECTCombine.
>>
>> vselects with constant masks, after legalization, will get turned into
>> specialized shuffle_vectors so they can be matched to blend+imm
>> instructions.
>>
>> Fixed some tests.Properties:
>>   phase_id: r209045-t20140516_160242-b19995
>>
>>
>>
>> File: test/Analysis/CostModel/X86/vselect-cost.ll
>> On: smooshlab-project
>> At: Fri 16 May 2014 16:00:20
>> Changed By: filcab
>> Comments: Added tests for the cost of lowering VSELECT instructions.Properties:
>>   phase_id: r209045-t20140516_160242-b19995
>>
>>
>>
>> Files:
>>  include/clang/AST/ASTContext.h
>>  include/clang/AST/Decl.h
>>  include/clang/AST/DeclCXX.h
>>  include/clang/AST/DeclObjC.h
>>  include/clang/AST/DeclTemplate.h
>>  include/clang/AST/ExternalASTSource.h
>>  include/clang/AST/Redeclarable.h
>>  include/clang/Sema/MultiplexExternalSemaSource.h
>>  include/clang/Serialization/ASTReader.h
>>  lib/AST/ASTContext.cpp
>>  lib/AST/Decl.cpp
>>  lib/AST/DeclCXX.cpp
>>  lib/AST/DeclObjC.cpp
>>  lib/AST/DeclTemplate.cpp
>>  lib/AST/ExternalASTSource.cpp
>>  lib/CodeGen/CGBlocks.cpp
>>  lib/CodeGen/CGDeclCXX.cpp
>>  lib/CodeGen/CGObjC.cpp
>>  lib/Frontend/FrontendAction.cpp
>>  lib/Sema/MultiplexExternalSemaSource.cpp
>>  lib/Serialization/ASTReader.cpp
>>  lib/Serialization/ASTReaderDecl.cpp
>>  lib/Serialization/ASTWriter.cpp
>>  test/Modules/Inputs/module.map
>>  test/Modules/Inputs/redecl-add-after-load.h
>>  test/Modules/decldef.mm
>>  test/Modules/redecl-add-after-load.cpp
>> On: smooshlab-project
>> At: Fri 16 May 2014 16:10:20
>> Changed By: rsmith
>> Comments: If a declaration is loaded, and then a module import adds a redeclaration, then
>> ensure that querying the first declaration for its most recent declaration
>> checks for redeclarations from the imported module.
>>
>> This works as follows:
>>  * The 'most recent' pointer on a canonical declaration grows a pointer to the
>>    external AST source and a generation number (space- and time-optimized for
>>    the case where there is no external source).
>>  * Each time the 'most recent' pointer is queried, if it has an external source,
>>    we check whether it's up to date, and update it if not.
>>  * The ancillary data stored on the canonical declaration is allocated lazily
>>    to avoid filling it in for declarations that end up being non-canonical.
>>    We'll still perform a redundant (ASTContext) allocation if someone asks for
>>    the most recent declaration from a decl before setPreviousDecl is called,
>>    but such cases are probably all bugs, and are now easy to find.
>>
>> Some finessing is still in order here -- in particular, we use a very general
>> mechanism for handling the DefinitionData pointer on CXXRecordData, and a more
>> targeted approach would be more compact.
>>
>> Also, the MayHaveOutOfDateDef mechanism should now be expunged, since it was
>> addressing only a corner of the full problem space here. That's not covered
>> by this patch.
>>
>> Early performance benchmarks show that this makes no measurable difference to
>> Clang performance without modules enabled (and fixes a major correctness issue
>> with modules enabled). I'll revert if a full performance comparison shows any
>> problems.
>> Properties:
>>   phase_id: r209048-t20140516_164908-b19998
>>
>>
>>
>> File: test/Modules/Inputs/redecl-add-after-load-top.h
>> On: smooshlab-project
>> At: Fri 16 May 2014 16:26:20
>> Changed By: rsmith
>> Comments: Add missed file from r209046.
>> Properties:
>>   phase_id: r209048-t20140516_164908-b19998
>>
>>
>>
>> Files:
>>  include/llvm/CodeGen/MachineOperand.h
>>  lib/Target/PowerPC/PPCAsmPrinter.cpp
>> On: smooshlab-project
>> At: Fri 16 May 2014 16:47:03
>> Changed By: pete
>> Comments: Use a sized enum for MachineOperandType.  No functionality changeProperties:
>>   phase_id: r209048-t20140516_164908-b19998
>>
>>
>>
>> Files:
>>  lib/Transforms/InstCombine/InstructionCombining.cpp
>>  test/Transforms/InstCombine/gepphigep.ll
>> On: smooshlab-project
>> At: Fri 16 May 2014 16:56:36
>> Changed By: louis
>> Comments: Add support for combining GEPs across PHI nodes
>>
>> Currently LLVM will generally merge GEPs. This allows backends to use more
>> complex addressing modes. In some cases this is not happening because there
>> is PHI inbetween the two GEPs:
>>
>>   GEP1--\
>>         |-->PHI1-->GEP3
>>   GEP2--/
>>
>> This patch checks to see if GEP1 and GEP2 are similiar enough that they can be
>> cloned (GEP12) in GEP3's BB, allowing GEP->GEP merging (GEP123):
>>
>>   GEP1--\                     --\                           --\
>>         |-->PHI1-->GEP3  ==>    |-->PHI2->GEP12->GEP3 == >    |-->PHI2->GEP123
>>   GEP2--/                     --/                           --/
>>
>> This also breaks certain use chains that are preventing GEP->GEP merges that the
>> the existing instcombine would merge otherwise.
>>
>> Tests included.
>>
>> rdar://15547484Properties:
>>   phase_id: r209053-t20140516_184122-b20003
>>
>>
>>
>> Files:
>>  docs/CommandGuide/llvm-symbolizer.rst
>>  include/llvm/DebugInfo/DIContext.h
>>  lib/DebugInfo/DWARFDebugInfoEntry.cpp
>>  test/DebugInfo/llvm-symbolizer.test
>>  tools/llvm-symbolizer/LLVMSymbolize.cpp
>>  tools/llvm-symbolizer/LLVMSymbolize.h
>>  tools/llvm-symbolizer/llvm-symbolizer.cpp
>> On: smooshlab-project
>> At: Fri 16 May 2014 17:20:23
>> Changed By: samsonov
>> Comments: [DWARF parser] Teach DIContext to fetch short (non-linkage) function names for a given address.
>>
>> Change --functions option in llvm-symbolizer tool to accept
>> values "none", "short" or "linkage". Update the tests and docs
>> accordingly.
>> Properties:
>>   phase_id: r209053-t20140516_184122-b20003
>>
>>
>>
>> File: source/Plugins/Disassembler/llvm/DisassemblerLLVMC.cpp
>> On: smooshlab-project
>> At: Fri 16 May 2014 17:40:20
>> Changed By: jmolenda
>> Comments: Update how we create our MCSymbolizer to keep working correctly
>> on arm64 binaries after the llvm r206063 changes.
>> Patch written by Jim Ingham and Lang Hames.
>> <rdar://problem/16935671>
>> Properties:
>>   phase_id: r209053-t20140516_184122-b20003
>>
>>
>>
>> Files:
>>  include/clang/Basic/DiagnosticSemaKinds.td
>>  lib/Sema/SemaChecking.cpp
>>  lib/Sema/SemaInit.cpp
>>  test/CXX/dcl.decl/dcl.init/dcl.init.list/basic.cpp
>>  test/CXX/dcl.decl/dcl.init/dcl.init.list/p3-0x.cpp
>>  test/CXX/dcl.decl/dcl.init/dcl.init.list/p7-0x-fixits.cpp
>>  test/CXX/dcl.decl/dcl.init/dcl.init.list/p7-0x.cpp
>>  test/CXX/dcl.decl/dcl.init/dcl.init.list/p7-cxx11-nowarn.cpp
>>  test/SemaCXX/cxx0x-initializer-constructor.cpp
>>  test/SemaCXX/cxx0x-initializer-scalars.cpp
>>  test/SemaCXX/cxx1y-variable-templates_top_level.cpp
>>  test/SemaCXX/microsoft-cxx0x.cpp
>>  test/SemaTemplate/constexpr-instantiate.cpp
>> On: smooshlab-project
>> At: Fri 16 May 2014 18:26:25
>> Changed By: alp
>> Comments: Tweak diagnostic wording for init list narrowing
>>
>> The conventional form is '<action> to silence this warning'.
>>
>> Also call the diagnostic an 'issue' rather than a 'message' because the latter
>> term is more widely used with reference to message expressions.Properties:
>>   phase_id: r209053-t20140516_184122-b20003
>>
>>
>>
>> Files:
>>  lib/profile/InstrProfiling.h
>>  lib/profile/InstrProfilingFile.c
>>  lib/profile/InstrProfilingRuntime.cc
>>  test/profile/Inputs/instrprof-dlopen-func.c
>>  test/profile/Inputs/instrprof-dlopen-func2.c
>>  test/profile/Inputs/instrprof-dlopen-main.c
>>  test/profile/instrprof-dlopen.test
>> On: smooshlab-project
>> At: Fri 16 May 2014 18:36:25
>> Changed By: dexonsmith
>> Comments: InstrProf: Support profiling dlopen'd shared libraries
>>
>> Shared objects are hard.  After this commit, we do the right thing when
>> profiling two separate shared objects that have been dlopen'd with
>> `RTLD_LOCAL`, when the main executable is *not* being profiled.
>>
>> This mainly simplifies the writer logic.
>>
>>   - At initialization, determine the output filename and truncate the
>>     file.  Depending on whether shared objects can see each other, this
>>     may happen multiple times.
>>
>>   - At exit, each executable writes its own profile in append mode.
>>
>> <rdar://problem/16918688>Properties:
>>   phase_id: r209053-t20140516_184122-b20003
>>
>>
>>
>> Files:
>>  lib/Sema/SemaOverload.cpp
>>  test/CXX/drs/dr4xx.cpp
>>  test/SemaCXX/overloaded-operator.cpp
>>  www/cxx_dr_status.html
>> On: smooshlab-project
>> At: Fri 16 May 2014 19:10:21
>> Changed By: rsmith
>> Comments: Correct incoherent function versus function template partial ordering for conversion operators (the comparison could claim that two conversion operators are both better than each other). Actually implement DR495, rather than passing its test by chance because the declarations happened to be in the "lucky" order.
>> Properties:
>>   phase_id: r209056-t20140516_204240-b20006
>>
>>
>>
>> File: lib/Sema/SemaOverload.cpp
>> On: smooshlab-project
>> At: Fri 16 May 2014 19:30:20
>> Changed By: rsmith
>> Comments: Switch to a more efficient expression of this check.
>> Properties:
>>   phase_id: r209056-t20140516_204240-b20006
>>
>>
>>
>> Files:
>>  lib/Target/X86/X86ISelLowering.cpp
>>  test/CodeGen/X86/sse3.ll
>> On: smooshlab-project
>> At: Fri 16 May 2014 20:40:36
>> Changed By: chandlerc
>> Comments: [x86] Fix a bad predicate I spotted by inspection -- pshufhw and pshuflw
>> were added in SSE2, no SSSE3. Found this while auditing all uses of
>> SSSE3 in the X86 target. I don't actually expect this to make
>> a significant difference on anything and I don't have any detailed test
>> cases but I updated the existing test cases that already covered some of
>> this code path.Properties:
>>   phase_id: r209056-t20140516_204240-b20006
>>
>>
>>
>> Files:
>>  lib/Target/ARM/ARMISelLowering.cpp
>>  test/CodeGen/ARM/Windows/read-only-data.ll
>> On: smooshlab-project
>> At: Fri 16 May 2014 21:40:21
>> Changed By: compnerd
>> Comments: ARM: use the proper target object format for WoA
>>
>> WoA uses COFF, not ELF.  ARMISelLowering::createTLOF would previously return ELF
>> for any non-MachO platform.  This was a missed site when the original change for
>> target format support for Windows on ARM was done.Properties:
>>   phase_id: r209060-t20140516_215339-b20008
>>
>>
>>
>> File: test/CodeGen/ARM/Windows/read-only-data.ll
>> On: smooshlab-project
>> At: Fri 16 May 2014 21:40:21
>> Changed By: compnerd
>> Comments: test: fix copy-paste mistake
>>
>> Accidental over-quoting of the match string.Properties:
>>   phase_id: r209060-t20140516_215339-b20008
>>
>>
>>
>> Files:
>>  lib/FrontendTool/ExecuteCompilerInvocation.cpp
>>  test/ARCMT/migrate-on-pch-and-module.m
>> On: smooshlab-project
>> At: Fri 16 May 2014 21:46:21
>> Changed By: akirtzidis
>> Comments: [objcmt] Don't wrap a PCH generation action with a ObjCMigrateAction one, it messes up the PCH file.
>>
>> rdar://16941811Properties:
>>   phase_id: r209060-t20140516_215339-b20008
>>
>>
>>
>> Files:
>>  lib/Sema/SemaOverload.cpp
>>  test/CXX/expr/expr.prim/expr.prim.lambda/blocks.mm
>> On: smooshlab-project
>> At: Fri 16 May 2014 21:46:21
>> Changed By: rsmith
>> Comments: More fixes for isBetterOverloadCandidate not being a strict weak ordering. The
>> bug was obvious from inspection, figuring out a way to test it was... less so.
>> Properties:
>>   phase_id: r209060-t20140516_215339-b20008
>>
>>
>>
>> Files:
>>  include/clang/Lex/Token.h
>>  lib/ARCMigrate/Transforms.cpp
>>  lib/Lex/Lexer.cpp
>>  lib/Lex/ModuleMap.cpp
>>  lib/Lex/PPDirectives.cpp
>>  lib/Lex/Preprocessor.cpp
>>  lib/Sema/SemaObjCProperty.cpp
>>  tools/libclang/CIndex.cpp
>> On: smooshlab-project
>> At: Fri 16 May 2014 22:06:20
>> Changed By: alp
>> Comments: Provide and use a safe Token::getRawIdentifier() accessorProperties:
>>   phase_id: r209064-t20140516_223324-b20010
>>
>>
>>
>> File: clang-tidy/google/GoogleTidyModule.cpp
>> On: smooshlab-project
>> At: Fri 16 May 2014 22:06:20
>> Changed By: alp
>> Comments: Track clang changes from r209061Properties:
>>   phase_id: r209064-t20140516_223324-b20010
>>
>>
>>
>> File: lib/Target/TargetLoweringObjectFile.cpp
>> On: smooshlab-project
>> At: Fri 16 May 2014 22:30:22
>> Changed By: majnemer
>> Comments: Target: Replace getSection().empty() with hasSection()
>>
>> No functional change, just a small cleanup.Properties:
>>   phase_id: r209064-t20140516_223324-b20010
>>
>>
>>
>> File: lib/Transforms/InstCombine/InstructionCombining.cpp
>> On: smooshlab-project
>> At: Sat 17 May 2014 00:00:23
>> Changed By: louis
>> Comments: Fix for sanitizer crash introduced in r209049
>>
>> This patch fixes 3 issues introduced by r209049 that only showed up in on
>> the sanitizer buildbots. One was a typo in a compare. The other is a check to
>> confirm that the single differing value in the two incoming GEPs is the same
>> type. The final issue was the the IRBuilder under some circumstances would
>> build PHIs in the middle of the block.Properties:
>>   phase_id: r209065-t20140517_000246-b20011
>>
>>
>>
>> File: include/llvm/Support/MachO.h
>> On: smooshlab-project
>> At: Sat 17 May 2014 00:46:28
>> Changed By: majnemer
>> Comments: MachO: Add comments describing section flags/attrs
>>
>> When pruning superfluous MachO structure definitions, I chose to keep
>> the most generically useful which is why Support's definitions won over
>> the ones in MC.
>>
>> However, the MC copy had some useful comments describing some of the
>> field values.
>>
>> Bring these back to the copy in Support.  While doing this, fill in some
>> of the underdocumented definitions as well.Properties:
>>   phase_id: r209066-t20140517_004848-b20012
>>
>>
>>
>> Files:
>>  lib/Transforms/InstCombine/InstructionCombining.cpp
>>  test/Transforms/InstCombine/gepphigep.ll
>> On: smooshlab-project
>> At: Sat 17 May 2014 07:50:20
>> Changed By: chapuni
>> Comments: Revert r209049 and r209065, "Add support for combining GEPs across PHI nodes"
>>
>> It broke clang selfhosting even after r209065.Properties:
>>   phase_id: r209067-t20140517_075227-b20013
>>
>>
>>
>> LOGS:
>>
>>




More information about the llvm-testresults mailing list