Skip to content

[ownership] Add a frontend option to stop optimizing right before we lower ownership. #33984

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

Conversation

gottesmm
Copy link
Contributor

Specifically the option: -sil-stop-optzns-before-lowering-ownership. This makes
it possible to write end-to-end tests on OSSA passes. Before one would have to
pattern match after ownership was lowered, losing the ability to do finegrained
FileCheck pattern matching on ossa itself.

…lower ownership.

Specifically the option: -sil-stop-optzns-before-lowering-ownership. This makes
it possible to write end-to-end tests on OSSA passes. Before one would have to
pattern match after ownership was lowered, losing the ability to do finegrained
FileCheck pattern matching on ossa itself.
@gottesmm
Copy link
Contributor Author

@swift-ci smoke test

@gottesmm gottesmm merged commit a445079 into swiftlang:master Sep 18, 2020
@gottesmm gottesmm deleted the pr-5adf3ac760b4c158f5710c5ddb5d7859625d8c4d branch September 18, 2020 03:38
gottesmm added a commit to gottesmm/swift that referenced this pull request Sep 18, 2020
I had this originally before landing swiftlang#33984, but it got merged while rebasing
into the wrong patch by mistake. Upstreaming ti now as a result.
gottesmm added a commit that referenced this pull request Sep 20, 2020
…d453ce40de03aa

[gardening] Add a test case for #33984
ainu-bot added a commit to google/swift that referenced this pull request Sep 20, 2020
* 'master' of github.com:apple/swift:
  [gardening] Add a test case for swiftlang#33984
meg-gupta pushed a commit to meg-gupta/swift that referenced this pull request Sep 21, 2020
I had this originally before landing swiftlang#33984, but it got merged while rebasing
into the wrong patch by mistake. Upstreaming ti now as a result.
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