Skip to content

[ome] Just erase the destructure rather than trying to get rid of dea… #27599

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

…d instructions due to simplifyInstruction.

This ensures that we do not run into any iterator invalidation issues. It does
worsen the passes output slightly, but avoiding the complexity is worth it and
other later passes will clean up the dead code.

rdar://55811732

@gottesmm gottesmm requested a review from eeckstein October 10, 2019 06:43
@gottesmm
Copy link
Contributor Author

@swift-ci test

Copy link
Contributor

@eeckstein eeckstein left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Code changes LGTM.

Can you add a test case (I think you had one)?

@gottesmm
Copy link
Contributor Author

@eeckstein I do! Sorry I didn't add it to the PR! Thanks for catching that!

…d instructions due to simplifyInstruction.

This ensures that we do not run into any iterator invalidation issues. It does
worsen the passes output slightly, but avoiding the complexity is worth it and
other later passes will clean up the dead code.

rdar://55811732
@gottesmm gottesmm force-pushed the pr-b367ebec31812f3fe81f71280f06107d97e18318 branch from 7a2b250 to 1b29d86 Compare October 10, 2019 16:07
@gottesmm
Copy link
Contributor Author

@swift-ci smoke test

2 similar comments
@gottesmm
Copy link
Contributor Author

@swift-ci smoke test

@gottesmm
Copy link
Contributor Author

@swift-ci smoke test

@gottesmm gottesmm merged commit 19b55ed into swiftlang:master Oct 10, 2019
@gottesmm gottesmm deleted the pr-b367ebec31812f3fe81f71280f06107d97e18318 branch October 10, 2019 17:24
@gottesmm gottesmm restored the pr-b367ebec31812f3fe81f71280f06107d97e18318 branch October 10, 2019 17:24
@gottesmm gottesmm deleted the pr-b367ebec31812f3fe81f71280f06107d97e18318 branch October 10, 2019 17:24
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