Skip to content

[5.9] PerformanceInliner: protect against misuse of @inline(__always) #64717

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
Mar 29, 2023

Conversation

eeckstein
Copy link
Contributor

@eeckstein eeckstein commented Mar 29, 2023

Inline-always should only be used on relatively small functions. It must not be used on recursive functions. Add a check that prevents that inlining of large @inline(__always) functions.

#64319
rdar://106655649

This is a cherry-pick of commit 0a4b0a8 of #64635

Inline-always should only be used on relatively small functions. It must not be used on recursive functions.
Add a check that prevents that inlining of large @inline(__always) functions.

swiftlang#64319
rdar://106655649
@eeckstein eeckstein requested a review from a team as a code owner March 29, 2023 12:27
@eeckstein
Copy link
Contributor Author

@swift-ci test

@eeckstein eeckstein merged commit 5292d6d into swiftlang:release/5.9 Mar 29, 2023
@eeckstein eeckstein deleted the fix-inline-always-5.9 branch March 29, 2023 21:54
@AnthonyLatsis AnthonyLatsis added the 🍒 release cherry pick Flag: Release branch cherry picks label May 3, 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.

3 participants