Skip to content

Begin refactoring type variable binding code. #15128

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

Merged
merged 1 commit into from
Mar 11, 2018
Merged

Begin refactoring type variable binding code. #15128

merged 1 commit into from
Mar 11, 2018

Conversation

rudkx
Copy link
Contributor

@rudkx rudkx commented Mar 9, 2018

Move all the code specific to dealing with relational constraints out
into a separate function.

I expect to do more refactoring here so this is likely not the final
form for this code - just one step in the direction of breaking things
down into smaller logical pieces.

Move all the code specific to dealing with relational constraints out
into a separate function.

I expect to do more refactoring here so this is likely not the final
form for this code - just one step in the direction of breaking things
down into smaller logical pieces.
@rudkx
Copy link
Contributor Author

rudkx commented Mar 9, 2018

@swift-ci Please smoke test

@rudkx
Copy link
Contributor Author

rudkx commented Mar 9, 2018

@swift-ci Please test source compatibility

Copy link
Contributor

@xedin xedin left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Nice!

@rudkx
Copy link
Contributor Author

rudkx commented Mar 10, 2018

Hmm, @vedantk have any ideas on this Linux failure?

18:12:15 CMake Error at CMakeLists.txt:87 (message):
18:12:15   LLDB test compilers not specified.  Tests will not run
18:12:15 
18:12:15 
18:12:15 -- Configuring incomplete, errors occurred!

@vedantk
Copy link
Contributor

vedantk commented Mar 10, 2018

I've been seeing some build failures locally after the transition branch was merged into stable, but Fred hasn't. I think one of the failures can be resolved by wiping the build directory. @shahmishal is there some way we could do that on the bot?

@shahmishal
Copy link
Member

@swift-ci Please clean smoke test

@rudkx
Copy link
Contributor Author

rudkx commented Mar 10, 2018

Linux build timeout after 300 minutes

@rudkx
Copy link
Contributor Author

rudkx commented Mar 10, 2018

@swift-ci Please clean smoke test Linux platform

@rudkx rudkx merged commit a28b57e into swiftlang:master Mar 11, 2018
@rudkx rudkx deleted the handle-relational-constraints-in-bindings branch March 11, 2018 07:51
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants