Skip to content

[5.3] No dummy constraint in the public swiftinterface for SPI extensions #31897

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
May 20, 2020

Conversation

xymus
Copy link
Contributor

@xymus xymus commented May 19, 2020

Don't print the dummy constraint _ConstraintThatIsNotPartOfTheAPIOfThisLibrary for SPI extensions in the public swiftinterface.

Cherry-pick of #31896.

  • Explanation: The compiler prints _ConstraintThatIsNotPartOfTheAPIOfThisLibrary to replace private constraints on public type conformances in the swiftinterface. This mechanism printed SPI extensions in the public swiftinterface to add the dummy conformance when they should have been skipped and reserved for the private swiftinterface.
  • Scope: The change affects only extensions using the @_spi attribute.
  • Risk: Low
  • Testing: Added a reduced test of the case we encountered.
  • Resolves rdar://63352700
  • Reviewed by: @nkcsgexi

@xymus xymus added the r5.3 label May 19, 2020
@xymus xymus requested a review from a team as a code owner May 19, 2020 20:44
@xymus
Copy link
Contributor Author

xymus commented May 19, 2020

@swift-ci Please test

@xymus

This comment has been minimized.

@xymus
Copy link
Contributor Author

xymus commented May 19, 2020

@swift-ci please nominate

@swift-ci
Copy link
Contributor

Build failed
Swift Test Linux Platform
Git Sha - a0fa529

@xymus
Copy link
Contributor Author

xymus commented May 19, 2020

@swift-ci Please test Linux platform

@DougGregor DougGregor merged commit deefd5e into swiftlang:release/5.3 May 20, 2020
@xymus xymus deleted the fix-dummy-constraint-5.3 branch October 31, 2022 21:02
@AnthonyLatsis AnthonyLatsis added swift 5.3 🍒 release cherry pick Flag: Release branch cherry picks labels Jan 8, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
🍒 release cherry pick Flag: Release branch cherry picks swift 5.3
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants