<div dir="ltr">Thanks for the hint!<div>I don't really want to fix it now as it's a very late evening - should better work on that tomorrow with a fresh mind.</div></div><div class="gmail_extra"><br><br><div class="gmail_quote">

2013/10/9 Benjamin Kramer <span dir="ltr"><<a href="mailto:benny.kra@gmail.com" target="_blank">benny.kra@gmail.com</a>></span><br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">

<div class="im"><br>
On 08.10.2013, at 22:12, Timur Iskhodzhanov <<a href="mailto:timurrrr@google.com">timurrrr@google.com</a>> wrote:<br>
<br>
> Looks like my change, will revert now and investigate tomorrow.<br>
> This hadn't been caught by check-clang...<br>
<br>
</div>std::sort may compare an element with itself, should be easy to fix the assert to avoid that case.<br>
<br>
- Ben<br>
<div class="HOEnZb"><div class="h5">><br>
> 2013/10/9  <<a href="mailto:llvmlab-buildmaster@lab.llvm.org">llvmlab-buildmaster@lab.llvm.org</a>>:<br>
>> The Buildbot has detected a new failure on builder phase2 - living while building <a href="http://lab.llvm.org" target="_blank">lab.llvm.org</a>.<br>
>> Full details are available at:<br>
>> <a href="http://lab.llvm.org:8013/builders/phase2%20-%20living/builds/5953" target="_blank">http://lab.llvm.org:8013/builders/phase2%20-%20living/builds/5953</a><br>
>><br>
>> Buildbot URL: <a href="http://lab.llvm.org:8013/" target="_blank">http://lab.llvm.org:8013/</a><br>
>><br>
>> Buildslave for this Build: macpro1<br>
>><br>
>> Build Reason: scheduler<br>
>> Build Source Stamp: 192220<br>
>> Blamelist: ahatanak,arsenm,d0k,enrico,kremenek,mren,smatveev,timurrrr<br>
>><br>
>> BUILD FAILED: failed<br>
>><br>
>> sincerely,<br>
>> -The Buildbot<br>
>><br>
>><br>
>> ================================================================================<br>
>><br>
>> CHANGES:<br>
>> File: include/llvm/IR/IRBuilder.h<br>
>> On: smooshlab-project<br>
>> At: Tue 08 Oct 2013 10:50:30<br>
>> Changed By: d0k<br>
>> Comments: IRBuilder: Downgrade InsertPointGuard's instruction pointer to a raw pointer.<br>
>><br>
>> Sadly this loses the checking from AssertingVH, but apparently storing the<br>
>> end() of a BasicBlock into an AssertingVH has bad consequences as it's not<br>
>> really an instruction.Properties:<br>
>>  phase_id: r192215-t20131008_115329-b12499<br>
>><br>
>><br>
>><br>
>> File: lib/Sema/SemaDecl.cpp<br>
>> On: smooshlab-project<br>
>> At: Tue 08 Oct 2013 11:10:30<br>
>> Changed By: kremenek<br>
>> Comments: Remove transient code I did not mean to check in.Properties:<br>
>>  phase_id: r192215-t20131008_115329-b12499<br>
>><br>
>><br>
>><br>
>> Files:<br>
>> test/CodeGen/R600/insert_vector_elt.ll<br>
>> test/CodeGen/R600/si-annotate-cf-assertion.ll<br>
>> test/CodeGen/R600/store-vector-ptrs.ll<br>
>> On: smooshlab-project<br>
>> At: Tue 08 Oct 2013 11:10:30<br>
>> Changed By: arsenm<br>
>> Comments: Add some xfaild R600 tests.<br>
>><br>
>> These are bugs to fix later.Properties:<br>
>>  phase_id: r192215-t20131008_115329-b12499<br>
>><br>
>><br>
>><br>
>> Files:<br>
>> lib/Target/Mips/MipsLongBranch.cpp<br>
>> lib/Target/Mips/MipsMachineFunction.cpp<br>
>> On: smooshlab-project<br>
>> At: Tue 08 Oct 2013 11:20:30<br>
>> Changed By: ahatanak<br>
>> Comments: [mips] Simplify and optimize code.<br>
>><br>
>> No intended functionality change.<br>
>> Properties:<br>
>>  phase_id: r192215-t20131008_115329-b12499<br>
>><br>
>><br>
>><br>
>> File: include/llvm/DebugInfo.h<br>
>> On: smooshlab-project<br>
>> At: Tue 08 Oct 2013 11:50:35<br>
>> Changed By: mren<br>
>> Comments: Move DIRef::getName out of line.Properties:<br>
>>  phase_id: r192215-t20131008_115329-b12499<br>
>><br>
>><br>
>><br>
>> File: lib/CodeGen/AsmPrinter/DwarfCompileUnit.cpp<br>
>> On: smooshlab-project<br>
>> At: Tue 08 Oct 2013 11:50:35<br>
>> Changed By: mren<br>
>> Comments: Debug Info: take advantage of the existing CU::resolve.Properties:<br>
>>  phase_id: r192215-t20131008_115329-b12499<br>
>><br>
>><br>
>><br>
>> File: lib/sanitizer_common/sanitizer_stoptheworld_linux_libcdep.cc<br>
>> On: smooshlab-project<br>
>> At: Tue 08 Oct 2013 11:06:30<br>
>> Changed By: smatveev<br>
>> Comments: [sanitizer] Fix deadlock in StopTheWorld.<br>
>><br>
>> The tracer thread in StopTheWorld could wait on a mutex forever if the parent<br>
>> process died before unlocking it. Use PR_SET_PDEATHSIG so that the parent would<br>
>> kill the child in this scenario.Properties:<br>
>>  phase_id: r192210-t20131008_120253-b12500<br>
>><br>
>><br>
>><br>
>> File: lib/CodeGen/AsmPrinter/DwarfCompileUnit.cpp<br>
>> On: smooshlab-project<br>
>> At: Tue 08 Oct 2013 11:56:30<br>
>> Changed By: mren<br>
>> Comments: Debug Info: rename getOriginalTypeSize to getBaseTypeSize.Properties:<br>
>>  phase_id: r192216-t20131008_121110-b12501<br>
>><br>
>><br>
>><br>
>> Files:<br>
>> include/lldb/DataFormatters/DataVisualization.h<br>
>> include/lldb/DataFormatters/FormatManager.h<br>
>> include/lldb/DataFormatters/TypeCategory.h<br>
>> include/lldb/DataFormatters/TypeCategoryMap.h<br>
>> include/lldb/lldb-private-enumerations.h<br>
>> source/API/SBTypeCategory.cpp<br>
>> source/Commands/CommandObjectType.cpp<br>
>> source/Core/ValueObject.cpp<br>
>> source/DataFormatters/DataVisualization.cpp<br>
>> source/DataFormatters/FormatManager.cpp<br>
>> source/DataFormatters/TypeCategory.cpp<br>
>> source/DataFormatters/TypeCategoryMap.cpp<br>
>> source/DataFormatters/TypeFormat.cpp<br>
>> On: smooshlab-project<br>
>> At: Tue 08 Oct 2013 12:10:30<br>
>> Changed By: enrico<br>
>> Comments: <rdar://problem/11778815><br>
>><br>
>> Formats (as in "type format") are now included in categories<br>
>> The only bit missing is caching formats along with synthetic children and summaries, which might be now desirable<br>
>><br>
>> Properties:<br>
>>  phase_id: r192217-t20131008_121929-b12502<br>
>><br>
>><br>
>><br>
>> Files:<br>
>> lib/CodeGen/AsmPrinter/DwarfDebug.cpp<br>
>> lib/CodeGen/AsmPrinter/DwarfDebug.h<br>
>> On: smooshlab-project<br>
>> At: Tue 08 Oct 2013 12:16:30<br>
>> Changed By: mren<br>
>> Comments: Add DbgVariable::resolve per Eric's suggestion.Properties:<br>
>>  phase_id: r192220-t20131008_122656-b12503<br>
>><br>
>><br>
>><br>
>> File: test/CodeGen/2008-01-25-ByValReadNone.c<br>
>> On: smooshlab-project<br>
>> At: Tue 08 Oct 2013 12:16:30<br>
>> Changed By: ahatanak<br>
>> Comments: [mips] Remove XFAIL.<br>
>><br>
>> Triple x86_64 has been added to the command line, so this test doesn't fail<br>
>> on mips anymore.<br>
>> Properties:<br>
>>  phase_id: r192220-t20131008_122656-b12503<br>
>><br>
>><br>
>><br>
>> Files:<br>
>> include/clang/AST/VTableBuilder.h<br>
>> include/clang/Basic/ABI.h<br>
>> lib/AST/MicrosoftMangle.cpp<br>
>> lib/AST/VTableBuilder.cpp<br>
>> lib/CodeGen/CGCXXABI.h<br>
>> lib/CodeGen/CGVTables.cpp<br>
>> lib/CodeGen/CGVTables.h<br>
>> lib/CodeGen/ItaniumCXXABI.cpp<br>
>> lib/CodeGen/MicrosoftCXXABI.cpp<br>
>> test/CodeGenCXX/microsoft-abi-thunks.cpp<br>
>> test/CodeGenCXX/microsoft-abi-virtual-inheritance.cpp<br>
>> test/CodeGenCXX/microsoft-abi-vtables-multiple-nonvirtual-inheritance.cpp<br>
>> test/CodeGenCXX/microsoft-abi-vtables-virtual-inheritance.cpp<br>
>> On: smooshlab-project<br>
>> At: Tue 08 Oct 2013 12:20:30<br>
>> Changed By: timurrrr<br>
>> Comments: Abstract out parts of thunk emission code, add support for simple thunks when using -cxx-abi microsoft<br>
>><br>
>> Reviewed at <a href="http://llvm-reviews.chandlerc.com/D1787Properties" target="_blank">http://llvm-reviews.chandlerc.com/D1787Properties</a>:<br>
>>  phase_id: r192220-t20131008_122656-b12503<br>
>><br>
>><br>
>><br>
>> LOGS:<br>
>><br>
>><br>
<br>
</div></div></blockquote></div><br></div>