Skip to content

[backport: 5.2] Add _mangledTypeName to allow round trips T->mangledName->T #30576

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

Closed

Conversation

ktoso
Copy link
Contributor

@ktoso ktoso commented Mar 23, 2020

Backport of Add _mangledTypeName to allow round trips T->mangledName->T #30318

The patch also applies cleanly on 5.1, so I'm wondering if we want to backport there as well...
I think personally I'm happy enough with 5.2 and asking users to upgrade.

cc @jckarter

@ktoso ktoso requested a review from a team as a code owner March 23, 2020 02:48
@ktoso
Copy link
Contributor Author

ktoso commented Mar 23, 2020

@swift-ci test

@ktoso ktoso changed the title Add precondition that no repairs were made to _mangledTypeName string [backport: 5.2] Add precondition that no repairs were made to _mangledTypeName string Mar 23, 2020
@swift-ci
Copy link
Contributor

Build failed
Swift Test Linux Platform
Git Sha - 777a30ec23e53fce386b072a0d319b2c1cfd1b06

@swift-ci
Copy link
Contributor

Build failed
Swift Test OS X Platform
Git Sha - 777a30ec23e53fce386b072a0d319b2c1cfd1b06

@ktoso
Copy link
Contributor Author

ktoso commented Mar 24, 2020

Failure: Whoops, my bad, looking into it

@ktoso ktoso changed the title [backport: 5.2] Add precondition that no repairs were made to _mangledTypeName string [backport: 5.2] Add _mangledTypeName to allow round trips T->mangledName->T Mar 24, 2020
@ktoso ktoso force-pushed the backport-5.2-wip-mangledTypeName branch from 777a30e to f812dc3 Compare March 24, 2020 04:14
…ups)

Removing from backport as it's not a flag used on 5.2 it seems.
@ktoso
Copy link
Contributor Author

ktoso commented Mar 24, 2020

@swift-ci smoke test

@ktoso
Copy link
Contributor Author

ktoso commented Mar 24, 2020

@swift-ci test

@ktoso
Copy link
Contributor Author

ktoso commented Mar 31, 2020

Parking the 5.2 PR

@ktoso ktoso closed this Mar 31, 2020
@AnthonyLatsis AnthonyLatsis added swift 5.2 🍒 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
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants