Skip to content

Conversation

jckarter
Copy link
Contributor

Explanation: @_extern was accidentally enabled as a mainline feature even though it's experimental. This restores its experimental status.
Scope: Bug fix.
Issue: rdar://132013078
Original PR: #75334
Risk: Low. Projects shouldn't be using @_extern, but if they are, they have to -enable-experimental-feature Extern.
Testing: Swift CI
Reviewer: @tshortli

@jckarter jckarter requested a review from a team as a code owner July 18, 2024 16:20
@jckarter
Copy link
Contributor Author

@swift-ci Please test

@hborla
Copy link
Member

hborla commented Jul 18, 2024

@_extern is a new experimental feature in the 6.0 compiler, right? As in, no projects can be using this feature in production because there's no released compiler that supports it?

@jckarter
Copy link
Contributor Author

Yeah, it was added in 6.0, so it shouldn't have had an opportunity to be used in production yet. It looks like it was originally implemented as an EXPERIMENTAL_FEATURE and changed to a baseline feature by accident.

@jckarter jckarter force-pushed the extern-is-experimental-6.0 branch from ff09c9f to 596879d Compare July 18, 2024 16:57
@jckarter
Copy link
Contributor Author

@swift-ci Please test

@jckarter jckarter force-pushed the extern-is-experimental-6.0 branch from 596879d to 6fbefae Compare July 18, 2024 18:48
@jckarter
Copy link
Contributor Author

@swift-ci Please test

Mark it as such. rdar://132013078
@jckarter jckarter force-pushed the extern-is-experimental-6.0 branch from 6fbefae to 345e5a1 Compare July 18, 2024 20:35
@jckarter
Copy link
Contributor Author

@swift-ci Please test

@jckarter jckarter merged commit 1e34dbc into swiftlang:release/6.0 Jul 19, 2024
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.

2 participants