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

[mono] Enable branch optimizations when using LLVM. #93092

Merged
merged 1 commit into from
Oct 6, 2023

Conversation

vargaz
Copy link
Contributor

@vargaz vargaz commented Oct 6, 2023

Doing these optimizations early enables the removal of some dead code which decreases the amount of generic instances the aot compiler generates since the aot compiler will no longer encounter the methods called in the dead code.

@vargaz
Copy link
Contributor Author

vargaz commented Oct 6, 2023

This saves about 100k on a wasm hello world.

@vargaz
Copy link
Contributor Author

vargaz commented Oct 6, 2023

/azp run runtime-wasm

@azure-pipelines
Copy link

Azure Pipelines successfully started running 1 pipeline(s).

@vargaz
Copy link
Contributor Author

vargaz commented Oct 6, 2023

Failures are unrelated.

Doing these optimizations early enables the removal of some dead code which decreases
the amount of generic instances the aot compiler generates since the aot compiler
will no longer encounter the methods called in the dead code.
@vargaz
Copy link
Contributor Author

vargaz commented Oct 6, 2023

/azp run runtime-wasm

@azure-pipelines
Copy link

Azure Pipelines successfully started running 1 pipeline(s).

@vargaz
Copy link
Contributor Author

vargaz commented Oct 6, 2023

Failures are unrelated.

@vargaz vargaz merged commit 46c8741 into dotnet:main Oct 6, 2023
113 of 118 checks passed
@vargaz vargaz deleted the llvm-branch branch October 6, 2023 23:50
@ghost ghost locked as resolved and limited conversation to collaborators Nov 6, 2023
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants