Skip to content

ARCSequenceOpts: fix a wrong handling of applies with indirect out arguments #66221

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
May 31, 2023

Conversation

eeckstein
Copy link
Contributor

Fixes a miscompile

#64921
rdar://109920867

@eeckstein
Copy link
Contributor Author

@swift-ci test

@eeckstein
Copy link
Contributor Author

@swift-ci benchmark

@eeckstein eeckstein requested a review from gottesmm May 30, 2023 19:57
@tomerd
Copy link
Contributor

tomerd commented May 30, 2023

cc @weissi

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.

LGTM

@eeckstein eeckstein merged commit 36b1fee into swiftlang:main May 31, 2023
@eeckstein eeckstein deleted the fix-arc-sequence-opts branch May 31, 2023 08:22
eeckstein added a commit to eeckstein/swift that referenced this pull request May 31, 2023
…swift

Those tests are failing after fixing ARCSequenceOpts in swiftlang#66221

rdar://110055637
eeckstein added a commit to eeckstein/swift that referenced this pull request Jun 1, 2023
…swift

Those tests are failing after fixing ARCSequenceOpts in swiftlang#66221

rdar://110055637
eeckstein added a commit that referenced this pull request Jun 1, 2023
[5.9] ARCSequenceOpts: fix a wrong handling of applies with indirect out arguments #66221
meg-gupta pushed a commit to meg-gupta/swift that referenced this pull request Jun 2, 2023
…swift

Those tests are failing after fixing ARCSequenceOpts in swiftlang#66221

rdar://110055637
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.

3 participants