[Swift CI] Build Failure: 0. OSS - LLDB Incremental - OS X (master) #371

Development CI Notifications
21

Relevant part of the change:

Simplify SILGenApply some more by slavapestov · Pull Request #12787 · apple/swift · GitHub

The function_ref or class_method for a method calls now comes after argument evaluation.

Should it be fixed in LLDB or Swift?

I think it exposed an existing issue in Swift’s debug info emission.

Slava

···

On Nov 9, 2017, at 6:25 PM, Pavel Yaskevich <pyaskevich@apple.com> wrote:

On Nov 9, 2017, at 6:24 PM, Slava Pestov <spestov@apple.com <mailto:spestov@apple.com>> wrote:

Slava

On Nov 9, 2017, at 6:20 PM, Slava Pestov <spestov@apple.com <mailto:spestov@apple.com>> wrote:

I think it’s this change: Simplify SILGenApply some more by slavapestov · Pull Request #12787 · apple/swift · GitHub

It caused some SIL instructions to be re-ordered in an equivalent way.

22

Relevant part of the change:

Simplify SILGenApply some more by slavapestov · Pull Request #12787 · apple/swift · GitHub

The function_ref or class_method for a method calls now comes after argument evaluation.

Should it be fixed in LLDB or Swift?

I think it exposed an existing issue in Swift’s debug info emission.

Thanks for your help, guy! I’ve created rdar://problem/35459092 <rdar://problem/35459092> to track this.

···

On Nov 9, 2017, at 6:25 PM, Slava Pestov <spestov@apple.com> wrote:

On Nov 9, 2017, at 6:25 PM, Pavel Yaskevich <pyaskevich@apple.com <mailto:pyaskevich@apple.com>> wrote:

On Nov 9, 2017, at 6:24 PM, Slava Pestov <spestov@apple.com <mailto:spestov@apple.com>> wrote:

Slava

Slava

On Nov 9, 2017, at 6:20 PM, Slava Pestov <spestov@apple.com <mailto:spestov@apple.com>> wrote:

I think it’s this change: Simplify SILGenApply some more by slavapestov · Pull Request #12787 · apple/swift · GitHub

It caused some SIL instructions to be re-ordered in an equivalent way.

23

If this isn't going to get fixed/backed out quickly then we should disable this test temporarily and file another radar to re-enable it when the debug info is fixed.

Jim

···

On Nov 9, 2017, at 6:39 PM, Pavel Yaskevich via swift-lldb-dev <swift-lldb-dev@swift.org> wrote:

On Nov 9, 2017, at 6:25 PM, Slava Pestov <spestov@apple.com <mailto:spestov@apple.com>> wrote:

On Nov 9, 2017, at 6:25 PM, Pavel Yaskevich <pyaskevich@apple.com <mailto:pyaskevich@apple.com>> wrote:

On Nov 9, 2017, at 6:24 PM, Slava Pestov <spestov@apple.com <mailto:spestov@apple.com>> wrote:

Relevant part of the change:

Simplify SILGenApply some more by slavapestov · Pull Request #12787 · apple/swift · GitHub

The function_ref or class_method for a method calls now comes after argument evaluation.

Should it be fixed in LLDB or Swift?

I think it exposed an existing issue in Swift’s debug info emission.

Thanks for your help, guy! I’ve created rdar://problem/35459092 <rdar://problem/35459092> to track this.

Slava

Slava

On Nov 9, 2017, at 6:20 PM, Slava Pestov <spestov@apple.com <mailto:spestov@apple.com>> wrote:

I think it’s this change: Simplify SILGenApply some more by slavapestov · Pull Request #12787 · apple/swift · GitHub

It caused some SIL instructions to be re-ordered in an equivalent way.

_______________________________________________
swift-lldb-dev mailing list
swift-lldb-dev@swift.org
https://lists.swift.org/mailman/listinfo/swift-lldb-dev

24

Adrian is looking at the same issue I think — rdar://35430708 <rdar://35430708>.

Slava

···

On Nov 9, 2017, at 6:54 PM, Jim Ingham <jingham@apple.com> wrote:

If this isn't going to get fixed/backed out quickly then we should disable this test temporarily and file another radar to re-enable it when the debug info is fixed.

Jim

On Nov 9, 2017, at 6:39 PM, Pavel Yaskevich via swift-lldb-dev <swift-lldb-dev@swift.org <mailto:swift-lldb-dev@swift.org>> wrote:

On Nov 9, 2017, at 6:25 PM, Slava Pestov <spestov@apple.com <mailto:spestov@apple.com>> wrote:

On Nov 9, 2017, at 6:25 PM, Pavel Yaskevich <pyaskevich@apple.com <mailto:pyaskevich@apple.com>> wrote:

On Nov 9, 2017, at 6:24 PM, Slava Pestov <spestov@apple.com <mailto:spestov@apple.com>> wrote:

Relevant part of the change:

Simplify SILGenApply some more by slavapestov · Pull Request #12787 · apple/swift · GitHub

The function_ref or class_method for a method calls now comes after argument evaluation.

Should it be fixed in LLDB or Swift?

I think it exposed an existing issue in Swift’s debug info emission.

Thanks for your help, guy! I’ve created rdar://problem/35459092 <rdar://problem/35459092> to track this.

Slava

Slava

On Nov 9, 2017, at 6:20 PM, Slava Pestov <spestov@apple.com <mailto:spestov@apple.com>> wrote:

I think it’s this change: Simplify SILGenApply some more by slavapestov · Pull Request #12787 · apple/swift · GitHub

It caused some SIL instructions to be re-ordered in an equivalent way.

_______________________________________________
swift-lldb-dev mailing list
swift-lldb-dev@swift.org <mailto:swift-lldb-dev@swift.org>
https://lists.swift.org/mailman/listinfo/swift-lldb-dev

25

Adrian is looking at the same issue I think — rdar://35430708 <rdar://35430708>.

Slava

If this isn't going to get fixed/backed out quickly then we should disable this test temporarily and file another radar to re-enable it when the debug info is fixed.

I’m all for disabling the test until this gets resolved.

···

On Nov 9, 2017, at 6:54 PM, Slava Pestov <spestov@apple.com> wrote:

On Nov 9, 2017, at 6:54 PM, Jim Ingham <jingham@apple.com <mailto:jingham@apple.com>> wrote:

Jim

On Nov 9, 2017, at 6:39 PM, Pavel Yaskevich via swift-lldb-dev <swift-lldb-dev@swift.org <mailto:swift-lldb-dev@swift.org>> wrote:

On Nov 9, 2017, at 6:25 PM, Slava Pestov <spestov@apple.com <mailto:spestov@apple.com>> wrote:

On Nov 9, 2017, at 6:25 PM, Pavel Yaskevich <pyaskevich@apple.com <mailto:pyaskevich@apple.com>> wrote:

On Nov 9, 2017, at 6:24 PM, Slava Pestov <spestov@apple.com <mailto:spestov@apple.com>> wrote:

Relevant part of the change:

Simplify SILGenApply some more by slavapestov · Pull Request #12787 · apple/swift · GitHub

The function_ref or class_method for a method calls now comes after argument evaluation.

Should it be fixed in LLDB or Swift?

I think it exposed an existing issue in Swift’s debug info emission.

Thanks for your help, guy! I’ve created rdar://problem/35459092 <rdar://problem/35459092> to track this.

Slava

Slava

On Nov 9, 2017, at 6:20 PM, Slava Pestov <spestov@apple.com <mailto:spestov@apple.com>> wrote:

I think it’s this change: Simplify SILGenApply some more by slavapestov · Pull Request #12787 · apple/swift · GitHub

It caused some SIL instructions to be re-ordered in an equivalent way.

_______________________________________________
swift-lldb-dev mailing list
swift-lldb-dev@swift.org <mailto:swift-lldb-dev@swift.org>
https://lists.swift.org/mailman/listinfo/swift-lldb-dev

26

Okay. I failed the test.

Jim

···

On Nov 9, 2017, at 6:54 PM, Slava Pestov via swift-lldb-dev <swift-lldb-dev@swift.org> wrote:

Adrian is looking at the same issue I think — rdar://35430708 <rdar://35430708>.

Slava

On Nov 9, 2017, at 6:54 PM, Jim Ingham <jingham@apple.com <mailto:jingham@apple.com>> wrote:

If this isn't going to get fixed/backed out quickly then we should disable this test temporarily and file another radar to re-enable it when the debug info is fixed.

Jim

On Nov 9, 2017, at 6:39 PM, Pavel Yaskevich via swift-lldb-dev <swift-lldb-dev@swift.org <mailto:swift-lldb-dev@swift.org>> wrote:

On Nov 9, 2017, at 6:25 PM, Slava Pestov <spestov@apple.com <mailto:spestov@apple.com>> wrote:

On Nov 9, 2017, at 6:25 PM, Pavel Yaskevich <pyaskevich@apple.com <mailto:pyaskevich@apple.com>> wrote:

On Nov 9, 2017, at 6:24 PM, Slava Pestov <spestov@apple.com <mailto:spestov@apple.com>> wrote:

Relevant part of the change:

Simplify SILGenApply some more by slavapestov · Pull Request #12787 · apple/swift · GitHub

The function_ref or class_method for a method calls now comes after argument evaluation.

Should it be fixed in LLDB or Swift?

I think it exposed an existing issue in Swift’s debug info emission.

Thanks for your help, guy! I’ve created rdar://problem/35459092 <rdar://problem/35459092> to track this.

Slava

Slava

On Nov 9, 2017, at 6:20 PM, Slava Pestov <spestov@apple.com <mailto:spestov@apple.com>> wrote:

I think it’s this change: Simplify SILGenApply some more by slavapestov · Pull Request #12787 · apple/swift · GitHub

It caused some SIL instructions to be re-ordered in an equivalent way.

_______________________________________________
swift-lldb-dev mailing list
swift-lldb-dev@swift.org <mailto:swift-lldb-dev@swift.org>
https://lists.swift.org/mailman/listinfo/swift-lldb-dev

_______________________________________________
swift-lldb-dev mailing list
swift-lldb-dev@swift.org
https://lists.swift.org/mailman/listinfo/swift-lldb-dev