Skip to content
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

Switch osu!framework iOS projects to use AOT compilation on release configurations #5903

Merged
merged 5 commits into from
Jul 7, 2023

Conversation

frenzibyte
Copy link
Member

@frenzibyte frenzibyte commented Jul 7, 2023

First step towards #5902. After recent investigations, turns out Mono interpreter isn't as efficient as AOT is, and is the cause of many minor overheads that are affecting draw time negatively.

Switching osu! to use AOT compilation is going to be a considerable effort so I'll change this on o!f first and begin investigating iOS performance on a higher scale before moving into osu!.

Note that we're intentionally not using the --interpreter=all flag here (which allows the app to fall back to interpretation for dynamic code rather than throw exception), as we don't want to rely on interpretation at all until we come across a case where we can't help it.

@peppy
Copy link
Member

peppy commented Jul 7, 2023

Note that we're intentionally not using the --interpreter=all flag here (which allows the app to fall back to interpretation for dynamic code rather than throw exception), as we don't want to rely on interpretation at all until we come across a case where we can't help it.

I believe this will mean switching signalr back to crap mode, which I'd rather avoid if we can (at least osu! side).

But I guess this is a per-project setting so that's fine?

@frenzibyte
Copy link
Member Author

Yeah we can set this flag locally in osu! if we want to.

@peppy peppy enabled auto-merge July 7, 2023 00:59
@peppy peppy merged commit 0871079 into ppy:master Jul 7, 2023
@frenzibyte frenzibyte deleted the ios-release-aot branch July 7, 2023 01:17
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants