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] Compile runtime with MBR support by default on Android and Browser #47785

Merged
merged 3 commits into from
Feb 4, 2021

Conversation

lambdageek
Copy link
Member

@lambdageek lambdageek commented Feb 2, 2021

Compile the runtime with MBR support by default on Android and Browser. Other platforms can continue to opt in with /p:MonoMetadataUpdate=true.

We did some perf investigations #46842 (comment) that showed that there is some impact but it was not clearly actionable.

@ghost
Copy link

ghost commented Feb 2, 2021

Tagging subscribers to this area: @CoffeeFlux
See info in area-owners.md if you want to be subscribed.

Issue Details

Compile the runtime with MBR support by default.

We did some perf investigations #46842 (comment) that showed that there is some impact but it was not clearly actionable.

Author: lambdageek
Assignees: -
Labels:

area-VM-meta-mono

Milestone: -

@lambdageek

This comment has been minimized.

@lambdageek lambdageek changed the title [mono] Compile runtime with MBR support by default if interpreter is enabled [mono] Compile runtime with MBR support by default on Android and Browser Feb 3, 2021
@lambdageek
Copy link
Member Author

Updated the PR to just enable MBR on Android and Browser

@lambdageek lambdageek mentioned this pull request Feb 3, 2021
51 tasks
src/mono/mono.proj Outdated Show resolved Hide resolved
src/mono/mono.proj Outdated Show resolved Hide resolved
@lambdageek lambdageek merged commit 9f31242 into dotnet:master Feb 4, 2021
@ghost ghost locked as resolved and limited conversation to collaborators Mar 6, 2021
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