Skip to content

"-swift-version 3" means Swift 3.1, not 3.0. #7883

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 3, 2017

Conversation

jrose-apple
Copy link
Contributor

Put in a general mechanism for mapping user-specified "compatibility versions" to proper "effective versions" (what #if and @available checking should respect). This may still be different from the intrinsic "language version"; right now master is considered a "3.1" compiler with a "Swift 4 mode", and we plan to ship a "4.0" compiler with a "Swift 3 mode" that will have a version number of something like "3.2".

rdar://problem/29884401 / SR-3791

@jrose-apple jrose-apple requested a review from graydon March 3, 2017 02:42
@jrose-apple
Copy link
Contributor Author

@swift-ci Please test

Copy link
Contributor

@graydon graydon left a comment

Choose a reason for hiding this comment

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

LGTM (I mean it all looks terrifying; but plausibly more-correct than before)

Put in a general mechanism for mapping user-specified "compatibility
versions" to proper "effective versions" (what #if and @available
checking should respect). This may still be different from the
intrinsic "language version"; right now master is considered a "3.1"
compiler with a "Swift 4 mode", and we plan to ship a "4.0" compiler
with a "Swift 3 mode" that will have a version number of something
like "3.2".

rdar://problem/29884401 / SR-3791
@jrose-apple jrose-apple force-pushed the 3-is-not-always-3.0 branch from 0d7c744 to beaca18 Compare March 3, 2017 18:50
@jrose-apple
Copy link
Contributor Author

Fixed a comment typo.

@swift-ci Please smoke test

@jrose-apple
Copy link
Contributor Author

Hm, hasBrokenDebugInfo again. I thought we fixed that everywhere? Or is it because this is an incremental bot? @shahmishal @adrian-prantl

@adrian-prantl
Copy link
Contributor

@swift-ci please smoke test

@adrian-prantl
Copy link
Contributor

@jrose-apple: I recomitted the offending LLVM check because (as I now know) I tested the wrong configuration locally... It's reverted again.

@slavapestov
Copy link
Contributor

What about Swift 3.11 for workgroups?

@jrose-apple jrose-apple merged commit 3456d04 into swiftlang:master Mar 3, 2017
@jrose-apple jrose-apple deleted the 3-is-not-always-3.0 branch March 3, 2017 21:28
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