Skip to content

Conversation

@eeckstein
Copy link
Contributor

Optimizations based on deployment targets do not interfere well with inlinable functions

rdar://87898692

Optimizations based on deployment targets do not interfere well with inlinable functions

rdar://87898692
@eeckstein eeckstein added the r5.6 label Jan 26, 2022
@eeckstein eeckstein requested a review from a team as a code owner January 26, 2022 15:58
@eeckstein
Copy link
Contributor Author

@swift-ci test

Copy link
Contributor

@aschwaighofer aschwaighofer left a comment

Choose a reason for hiding this comment

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

Looks good to me.

@swift-ci
Copy link
Contributor

Build failed
Swift Test Linux Platform
Git Sha - c589c05

@slavapestov
Copy link
Contributor

You should disable the optimization for serializable functions and leave it in place for others.

@eeckstein
Copy link
Contributor Author

@swift-ci test linux

@eeckstein
Copy link
Contributor Author

@swift-ci test

@tbkka
Copy link
Contributor

tbkka commented Jan 26, 2022

Note: a more accurate fix is being developed for the main branch in #41026. This quick fix is only for the 5.6 branch.

@eeckstein eeckstein merged commit 17dc994 into swiftlang:release/5.6 Jan 27, 2022
@eeckstein eeckstein deleted the disable-immortal-arc-elimination branch January 27, 2022 06:55
@AnthonyLatsis AnthonyLatsis added 🍒 release cherry pick Flag: Release branch cherry picks swift 5.6 labels Jan 8, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

🍒 release cherry pick Flag: Release branch cherry picks

Projects

None yet

Development

Successfully merging this pull request may close these issues.

7 participants