[Swift 2.2] IRGen: Don't try to bind witness tables for archetype protocol requirements that don't need them.


(Joe Groff) #1

Hey Joe, I'm testing a fix for a crash regression reported on Swift 2.2 as rdar://problem/24568361. I'd like to get this cherry-picked into the 2.2 branch.

https://github.com/apple/swift/pull/1245

-Joe


(Joe Pamer) #2

Thanks, Joe! The PR looks good to me, but would someone more familiar with IRGen mind taking a quick look? (Slava?)

- Joe

···

On Feb 9, 2016, at 1:08 PM, Joe Groff <jgroff@apple.com> wrote:

Hey Joe, I'm testing a fix for a crash regression reported on Swift 2.2 as rdar://problem/24568361. I'd like to get this cherry-picked into the 2.2 branch.

https://github.com/apple/swift/pull/1245

-Joe


(Slava Pestov) #3

I don't claim to have any kind of deep understanding of IRGen, but this change makes sense to me.

Slava

···

On Feb 9, 2016, at 1:12 PM, Joe Pamer <jpamer@apple.com> wrote:

Thanks, Joe! The PR looks good to me, but would someone more familiar with IRGen mind taking a quick look? (Slava?)

- Joe

On Feb 9, 2016, at 1:08 PM, Joe Groff <jgroff@apple.com> wrote:

Hey Joe, I'm testing a fix for a crash regression reported on Swift 2.2 as rdar://problem/24568361. I'd like to get this cherry-picked into the 2.2 branch.

https://github.com/apple/swift/pull/1245

-Joe


(Joe Pamer) #4

Your ringing endorsement, coupled with the fact that the CI tests are passing, means this change is approved for 2.2. Joe, please feel free to cherry-pick the change to swift-2.2-branch.

Thanks!
- Joe

···

On Feb 9, 2016, at 2:43 PM, Slava Pestov <spestov@apple.com> wrote:

I don't claim to have any kind of deep understanding of IRGen, but this change makes sense to me.

Slava

On Feb 9, 2016, at 1:12 PM, Joe Pamer <jpamer@apple.com> wrote:

Thanks, Joe! The PR looks good to me, but would someone more familiar with IRGen mind taking a quick look? (Slava?)

- Joe

On Feb 9, 2016, at 1:08 PM, Joe Groff <jgroff@apple.com> wrote:

Hey Joe, I'm testing a fix for a crash regression reported on Swift 2.2 as rdar://problem/24568361. I'd like to get this cherry-picked into the 2.2 branch.

https://github.com/apple/swift/pull/1245

-Joe