[swift-lldb-dev] [Swift CI] Build Failure: 0. OSS - LLDB Incremental - OS X (master) #230
Saleem Abdulrasool
compnerd at compnerd.org
Wed Oct 4 13:00:03 CDT 2017
On Wed, Oct 4, 2017 at 10:57 AM Adrian Prantl <aprantl at apple.com> wrote:
>
> On Oct 4, 2017, at 10:54 AM, Saleem Abdulrasool <compnerd at compnerd.org>
> wrote:
>
> Does the lldb test suite not run as part of the CI builds?
>
>
> AFAIK it is currently not running as part of pull request testing. (Which
> is obviously not ideal).
>
Getting some minimal coverage would be nice. But I think that would be
outside of the scope of my changes. That can also be done subsequently to
improve coverage.
>
> It seems that in the mean time, at least two new regressions have been
> introduced into the Windows build. That does make me slightly hesistent
> since it seems that it is more likely that other regressions will get
> introduced.
>
> In any case, I’m doing a build of lldb and will look into this once the
> build completes. Hopefully the failures also reproduces on Linux.
>
>
> Thanks! Jim (jingham at apple.com) should be able to help you with any
> LLDB-related questions.
>
Awesome! I might have to take you up on that offer :)
> -- adrian
>
>
> Saleem
>
> On Wed, Oct 4, 2017 at 10:36 AM Adrian Prantl <aprantl at apple.com> wrote:
>
>>
>> On Oct 4, 2017, at 10:33 AM, Saleem Abdulrasool <compnerd at compnerd.org>
>> wrote:
>>
>>
>> On Wed, Oct 4, 2017 at 10:29 AM Jim Ingham <jingham at apple.com> wrote:
>>
>>> Nothing significant has changed on the lldb side but we're getting a
>>> bunch of tests asserting here:
>>>
>>> 21:01:25
>>> Assertion failed: (isTypeMetadata()), function getDescription, file
>>> /Users/buildslave/jenkins/workspace/lldb-master-tools_RA/swift/include/swift/Runtime/Metadata.h,
>>> line 1469.
>>>
>>> Anybody got any clues of what might have changed on the Swift side to
>>> start tripping this assertion?
>>
>>
>> The type metadata layout has changed. The value witness table reference
>> is no longer an offset but rather a complete pointer.
>>
>> It is possible that lldb needs to be updated for that?
>>
>>
>> That sounds plausible. Would it be possible to revert the commit until a
>> solution is found? You can easily build lldb by checking it out next to
>> swift and adding "-l" to the build script.
>>
>> -- adrian
>>
>>
>>
>>> Jim
>>>
>>>
>>>
>>> > On Oct 3, 2017, at 9:38 PM, no-reply at swift.org wrote:
>>> >
>>> > [FAILURE] oss-lldb-incremental-osx [#230]
>>> >
>>> > Build URL: https://ci.swift.org/job/oss-lldb-incremental-osx/230/
>>> > Project: oss-lldb-incremental-osx
>>> > Date of build: Tue, 03 Oct 2017 22:55:41 -0500
>>> > Build duration: 43 min
>>> > Identified problems:
>>> >
>>> > • Regression test failed: This build failed because a regression
>>> test in the test suite FAILed. Below is a list of all errors:
>>> > • Indication 1
>>> > • Assertion failure: This build failed because of an assertion
>>> failure. Below is a list of all errors in the build log:
>>> > • Indication 1
>>> >
>>> > Changes
>>> >
>>> > • Commit 99e395b7e3f89b1f6f2874296a37add58aece6d6 by github:
>>> > [APINotes] Add 'RetainCountConvention' (#133)
>>> >
>>> > • edit: include/clang/APINotes/Types.h
>>> > • edit: lib/APINotes/APINotesWriter.cpp
>>> > • edit: test/APINotes/Inputs/roundtrip.apinotes
>>> > • edit: lib/APINotes/APINotesYAMLCompiler.cpp
>>> > • edit: lib/APINotes/APINotesReader.cpp
>>> > • add: test/APINotes/retain-count-convention.m
>>> > • edit:
>>> test/APINotes/Inputs/Frameworks/SimpleKit.framework/Headers/SimpleKit.h
>>> > • edit: lib/Sema/SemaAPINotes.cpp
>>> > • edit: test/APINotes/yaml-roundtrip.c
>>> > • edit:
>>> test/APINotes/Inputs/Frameworks/SimpleKit.framework/Headers/SimpleKit.apinotes
>>> >
>>> > • Commit 969421f5ac164d98f4650ae0f1a4fd10409cabad by aprantl:
>>> > [DebugInfo] Handle endianness when moving debug info for split integer
>>> >
>>> > • edit: lib/CodeGen/SelectionDAG/LegalizeTypes.cpp
>>> > • add: test/CodeGen/PowerPC/debuginfo-split-int.ll
>>> >
>>> > • Commit e77f221279851c2406e824e75ae8904ba7d89ab5 by aprantl:
>>> > Add a manpage for llvm-dwarfdump.
>>> >
>>> > • edit: docs/CMakeLists.txt
>>> > • edit: docs/CommandGuide/llvm-dwarfdump.rst
>>> >
>>> > • Commit 2850e656c89ac7c4837a01b57c52c036e88c81f7 by jingham:
>>> > This test is passing everywhere I can see, so I'm removing the xfail,
>>> to
>>> >
>>> > • edit:
>>> packages/Python/lldbsuite/test/functionalities/thread/exit_during_step/TestExitDuringStep.py
>>> >
>>> > • Commit 7e6b564bf551e1ccd4eaff2824d87dd89b71a686 by dgregor:
>>> > Add fixed crasher from rdar://problem/33575781
>>> >
>>> > • add:
>>> validation-test/compiler_crashers_2_fixed/0125-rdar33575781.swift
>>> >
>>> > • Commit 086c12114dfdff1b7b7179a6052f43a8d73557ed by compnerd:
>>> > IRGen: switch to absolute pointers for nominal type descriptors
>>> >
>>> > • edit: include/swift/Runtime/Metadata.h
>>> > • edit: lib/IRGen/ConstantBuilder.h
>>> > • edit: stdlib/public/runtime/Metadata.cpp
>>> > • edit: unittests/runtime/Metadata.cpp
>>> > • edit: lib/IRGen/GenMeta.cpp
>>> > • edit: stdlib/public/runtime/ProtocolConformance.cpp
>>> > • edit: test/IRGen/foreign_types.sil
>>> > • edit: test/IRGen/objc_attr_NSManaged.sil
>>> > • edit: include/swift/Remote/MetadataReader.h
>>> > • edit: stdlib/public/runtime/Casting.cpp
>>> > • edit: test/IRGen/field_type_vectors.sil
>>> >
>>> > • Commit 2645a6a4b9cf2df212da6db9a41b3363ff44de56 by dgregor:
>>> > [Deserialization] Configure protocol before loading requirement
>>> >
>>> > • add: test/Serialization/recursive_protocol_merge.swift
>>> > • add:
>>> test/Serialization/Inputs/recursive_protocol_merge_b.swift
>>> > • add:
>>> test/Serialization/Inputs/recursive_protocol_merge_a.swift
>>> > • edit: lib/Serialization/Deserialization.cpp
>>> >
>>> > • Commit 610aa582ce9647ad377812355d8ae36ec5cf0e31 by shajrawi:
>>> > Fixes (another) IRGen compiler crash caused by the new large types ABI
>>> >
>>> > • edit: test/IRGen/big_types_corner_cases.sil
>>> > • edit: lib/IRGen/LoadableByAddress.cpp
>>> >
>>> > • Commit 77554c1ae2b3b2de16b803409d9f7085ad186562 by ghoare:
>>> > [Stats] Fix typo.
>>> >
>>> > • edit: lib/Basic/Statistic.cpp
>>> >
>>> > • Commit 0e5b982d2561bc7c102e2abe984e19241735aaa2 by ghoare:
>>> > [Stats] Only use input filename, not mangled path, in stats file name.
>>> >
>>> > • edit: lib/Basic/Statistic.cpp
>>> >
>>> > • Commit c1a4bb490bcdf8d20c962ab43249eaee44ee037f by github:
>>> > [test] Define out part of this test that's crashing on Linux (#12258)
>>> >
>>> > • edit: test/ClangImporter/clang_builtins.swift
>>>
>>> --
>> Saleem Abdulrasool
>> compnerd (at) compnerd (dot) org
>>
>> --
> Saleem Abdulrasool
> compnerd (at) compnerd (dot) org
>
> --
Saleem Abdulrasool
compnerd (at) compnerd (dot) org
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <https://lists.swift.org/pipermail/swift-lldb-dev/attachments/20171004/4988611c/attachment.html>
More information about the swift-lldb-dev
mailing list