-
Notifications
You must be signed in to change notification settings - Fork 10.5k
Sema: Bind context generic params using the current ConstraintSystem's generic environment instead of the decl's DC. #9760
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Sema: Bind context generic params using the current ConstraintSystem's generic environment instead of the decl's DC. #9760
Conversation
@swift-ci Please smoke test |
@slavapestov Does this look good? |
Looks good to me (and for 4.0 branch as well) |
Looks like this resolved a compiler crasher too -- nice! |
…s generic environment instead of the decl's DC. Fixes SR-4833.
9ef400d
to
d16eb33
Compare
@swift-ci Please smoke test |
1 similar comment
@swift-ci Please smoke test |
@ddunbar This llbuild failure on Linux ring a bell?
|
The llbuild failure was just fixed by @neonichu in swiftlang/swift-llbuild#175 |
It was overlapped PRs landing, was fixed in swiftlang/swift-llbuild#175 |
@swift-ci please smoke test linux |
Thanks @aciidb0mb3r and @ddunbar! |
Fixes SR-4833.
rdar://problem/32134718