Skip to content

[CanonicalizeOSSALifetime] Bail early in lifetime extension backwards walk. #66680

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
merged 1 commit into from
Jun 16, 2023

Conversation

nate-chandler
Copy link
Contributor

While collecting originalLiveBlocks, walking backward from consuming blocks, if a visited block is already in originalLiveBlocks, don't visit its predecessors.

Continuing the backwards walk is wasteful.

rdar://110854874

While collecting originalLiveBlocks, walking backward from consuming
blocks, if a visited block is already in originalLiveBlocks, don't visit
its predecessors.

Continuing the backwards walk is wasteful.

rdar://110854874
@nate-chandler nate-chandler requested a review from atrick June 15, 2023 20:06
@nate-chandler
Copy link
Contributor Author

@swift-ci please test

@nate-chandler nate-chandler merged commit 49e9284 into swiftlang:main Jun 16, 2023
@nate-chandler nate-chandler deleted the rdar110854874 branch June 16, 2023 14:02
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.

1 participant