Skip to content

Revert "[Mangling] Uniformly use "So" for imported decls." #9233

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

Conversation

jrose-apple
Copy link
Contributor

This reverts commit 25985cb, from #8849. For now, we're trying to avoid non-essential non-structural changes to the mangling, so that the old mangling doesn't appear to change. That doesn't mean no changes at all, but we can save this one for later.

Goes with @eeckstein's #9228.

@jrose-apple jrose-apple requested a review from eeckstein May 3, 2017 18:17
@jrose-apple
Copy link
Contributor Author

Waiting for Erik's change to go in before running tests.

@jrose-apple jrose-apple force-pushed the revert-So-mangling-change branch from a73ff0e to 368128c Compare May 3, 2017 19:09
@jrose-apple
Copy link
Contributor Author

@swift-ci Please test

@swift-ci
Copy link
Contributor

swift-ci commented May 3, 2017

Build failed
Jenkins build - Swift Test Linux Platform
Git Commit - a73ff0e04dabf970cc5e68a8dfb1267be167169a
Test requested by - @jrose-apple

@swift-ci
Copy link
Contributor

swift-ci commented May 3, 2017

Build failed
Jenkins build - Swift Test OS X Platform
Git Commit - a73ff0e04dabf970cc5e68a8dfb1267be167169a
Test requested by - @jrose-apple

This reverts commit 25985cb. For now,
we're trying to avoid spurious non-structural changes to the mangling,
so that the /old/ mangling doesn't appear to change. That doesn't mean
no changes at all, but we can save this one for later.
@jrose-apple jrose-apple force-pushed the revert-So-mangling-change branch from 368128c to 8e0e91d Compare May 3, 2017 20:26
@jrose-apple
Copy link
Contributor Author

@swift-ci Please test

@swift-ci
Copy link
Contributor

swift-ci commented May 3, 2017

Build failed
Jenkins build - Swift Test OS X Platform
Git Commit - 368128c3be115c4c889868fb405662838e7d2007
Test requested by - @jrose-apple

@swift-ci
Copy link
Contributor

swift-ci commented May 3, 2017

Build failed
Jenkins build - Swift Test Linux Platform
Git Commit - 368128c3be115c4c889868fb405662838e7d2007
Test requested by - @jrose-apple

@jrose-apple
Copy link
Contributor Author

@swift-ci Please clean test Linux

@jrose-apple jrose-apple merged commit c8d3506 into swiftlang:master May 3, 2017
@jrose-apple jrose-apple deleted the revert-So-mangling-change branch May 3, 2017 23:13
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