Skip to content

[sil.rst] Add a discussion of "Dead End Blocks" and its implications upon OSSA. #35233

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

The main implication being that we allow for values to be leaked along dead end
blocks.

NOTE: For those who are unaware, a dead end block in SIL is a block that is
post-dominated in the same function by program terminating blocks (which we
model using unreachables).

@gottesmm gottesmm requested a review from atrick December 28, 2020 05:28
@gottesmm
Copy link
Contributor Author

@atrick I think you will like this one.

Copy link
Contributor

@atrick atrick left a comment

Choose a reason for hiding this comment

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

Thanks for doing this. See 2 comments below.

…upon OSSA.

The main implication being that we allow for values to be leaked along dead end
blocks.

NOTE: For those who are unaware, a dead end block in SIL is a block that is
post-dominated in the same function by program terminating blocks (which we
model using unreachables).
@gottesmm gottesmm force-pushed the pr-4542def9fc3d07b8054aa5ab5322083dc97ebe57 branch from 7afc519 to a7422b7 Compare December 29, 2020 21:33
@gottesmm
Copy link
Contributor Author

gottesmm commented Jan 4, 2021

@swift-ci smoke test

@gottesmm gottesmm merged commit 043acc8 into swiftlang:main Jan 5, 2021
@gottesmm gottesmm deleted the pr-4542def9fc3d07b8054aa5ab5322083dc97ebe57 branch January 5, 2021 02:06
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