Skip to content

SimplifyBeginCOWMutation: don't eliminate end_cow_mutation instructions with the [keep_unique] flag #70233

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
Dec 6, 2023

Conversation

eeckstein
Copy link
Contributor

This flag is important for following optimizations. Therefore such end_cow_mutation instructions must not be removed.

rdar://119178823

@eeckstein eeckstein requested a review from meg-gupta December 5, 2023 10:47
@eeckstein
Copy link
Contributor Author

@swift-ci test

@eeckstein
Copy link
Contributor Author

@swift-ci benchmark

Copy link
Contributor

@meg-gupta meg-gupta left a comment

Choose a reason for hiding this comment

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

LGTM

@eeckstein eeckstein force-pushed the fix-cow-simplification branch from eddc4ed to 0574610 Compare December 5, 2023 18:17
@eeckstein
Copy link
Contributor Author

@swift-ci smoke test

@eeckstein
Copy link
Contributor Author

@swift-ci smoke test linux

…ions with the `[keep_unique]` flag

This flag is important for following optimizations. Therefore such `end_cow_mutation` instructions must not be removed.
Also, keep the flag in a SILCombine transformation.

rdar://119178823
@eeckstein eeckstein force-pushed the fix-cow-simplification branch from 0574610 to c829d5b Compare December 6, 2023 08:01
@eeckstein
Copy link
Contributor Author

@swift-ci smoke test

@eeckstein
Copy link
Contributor Author

@swift-ci smoke test linux

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