Skip to content

Enable R2R when publishing with custom Configuration value for CoreCLR#11071

Draft
Copilot wants to merge 2 commits intomainfrom
copilot/enable-r2r-custom-configuration
Draft

Enable R2R when publishing with custom Configuration value for CoreCLR#11071
Copilot wants to merge 2 commits intomainfrom
copilot/enable-r2r-custom-configuration

Conversation

Copy link
Copy Markdown
Contributor

Copilot AI commented Apr 2, 2026

CoreCLR R2R was only enabled when Configuration == 'Release', so customers using custom configuration values (e.g. dotnet publish -c MyRelease) got no R2R optimization.

Changes

  • Microsoft.Android.Sdk.CoreCLR.targets: Add _IsPublishing check so R2R is enabled during any dotnet publish, regardless of configuration name:
    <PublishReadyToRun Condition=" '$(PublishReadyToRun)' == '' and ('$(Configuration)' == 'Release' or '$(_IsPublishing)' == 'true') ">true</PublishReadyToRun>
  • BuildTest2.cs: Add PublishReadyToRunCustomConfiguration test — creates a project with releaseConfigurationName: "CustomRelease" and _IsPublishing=true, then verifies the output assembly contains R2R data via PE header inspection.

Update the PublishReadyToRun condition in Microsoft.Android.Sdk.CoreCLR.targets
to also check _IsPublishing, so R2R is enabled either when Configuration is
Release OR when the project is being published via dotnet publish.

This fixes the scenario where customers use custom Configuration values
(e.g. dotnet publish -c MyCustomConfig) and R2R was not enabled for CoreCLR.

Fixes #9944

Agent-Logs-Url: https://github.com/dotnet/android/sessions/09eff3bb-5a27-4d30-9025-395f55206033

Co-authored-by: simonrozsival <374616+simonrozsival@users.noreply.github.com>
Copilot AI changed the title [WIP] Enable R2R when publishing with custom Configuration value Enable R2R when publishing with custom Configuration value for CoreCLR Apr 2, 2026
Copilot AI requested a review from simonrozsival April 2, 2026 09:33
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[CoreCLR] Enable R2R when publishing with custom Configuration value

2 participants