Skip to content

Enable tiering for async versions#129985

Merged
jakobbotsch merged 1 commit into
dotnet:mainfrom
jakobbotsch:fix-115096
Jun 30, 2026
Merged

Enable tiering for async versions#129985
jakobbotsch merged 1 commit into
dotnet:mainfrom
jakobbotsch:fix-115096

Conversation

@jakobbotsch

Copy link
Copy Markdown
Member

Seems like this may be all that's needed.

Fix #115096

@dotnet-policy-service

Copy link
Copy Markdown
Contributor

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

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

This PR updates CoreCLR’s tiered compilation eligibility rules to allow runtime async “async version” methods to participate in tiering (and thus become candidates for tiered PGO), while continuing to exclude wrapper-thunk async methods that are not supported.

Changes:

  • Relaxed the tiering exclusion for async thunk methods to allow those that SupportsAsyncVersionCodegen().
  • Updated the in-code comment describing async tiering behavior (though the wording could be made more precise; see review comments).

Comment thread src/coreclr/vm/method.cpp
Comment thread src/coreclr/vm/method.cpp
@jakobbotsch

Copy link
Copy Markdown
Member Author

PTAL @VSadov

@jakobbotsch
jakobbotsch requested a review from VSadov June 30, 2026 15:36
@VSadov

VSadov commented Jun 30, 2026

Copy link
Copy Markdown
Member

Seems like this may be all that's needed.

In theory why not, but there was a chance of some implementation detail to require adjustment.

Nice that it “just works”.

@jakobbotsch

Copy link
Copy Markdown
Member Author

/azp run runtime-coreclr outerloop, runtime-coreclr jitstress

@azure-pipelines

Copy link
Copy Markdown
Azure Pipelines successfully started running 2 pipeline(s).

@jakobbotsch

Copy link
Copy Markdown
Member Author

/ba-g Failure is #129876 and a timeout

@jakobbotsch
jakobbotsch merged commit 45e8d50 into dotnet:main Jun 30, 2026
187 of 193 checks passed
@jakobbotsch
jakobbotsch deleted the fix-115096 branch June 30, 2026 21:09
@dotnet-milestone-bot dotnet-milestone-bot Bot added this to the 11.0-preview7 milestone Jul 1, 2026
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.

[RuntimeAsync] PGO support.

4 participants