Skip to content

Document Async flag representation in IL#121281

Merged
jkotas merged 1 commit intomainfrom
dev/jcouv/runtime-async
Nov 5, 2025
Merged

Document Async flag representation in IL#121281
jkotas merged 1 commit intomainfrom
dev/jcouv/runtime-async

Conversation

@jcouv
Copy link
Member

@jcouv jcouv commented Nov 3, 2025

@agocke @VSadov for review
FYI @333fred

This was implemented: #115658

@jcouv jcouv self-assigned this Nov 3, 2025
Copilot AI review requested due to automatic review settings November 3, 2025 00:35
@github-actions github-actions bot added the needs-area-label An area label is needed to ensure this gets routed to the appropriate area owners label Nov 3, 2025
Copy link
Contributor

Copilot AI left a comment

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 adds documentation about the IL representation of the Async method flag to the runtime-async specification.

  • Adds clarification that the Async flag is represented by the async keyword in IL
  • Documents that ilasm and ildasm tools recognize this flag

@jkotas jkotas added runtime-async area-ILTools-coreclr and removed needs-area-label An area label is needed to ensure this gets routed to the appropriate area owners labels Nov 3, 2025
@dotnet-policy-service
Copy link
Contributor

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

@jkotas
Copy link
Member

jkotas commented Nov 4, 2025

@ilonatommy Could you please take a look why the Build Analysis is stuck and report-green is not kicking in?

@jkotas
Copy link
Member

jkotas commented Nov 5, 2025

/ba-g docs only change

@jkotas jkotas merged commit ad47f63 into main Nov 5, 2025
25 of 26 checks passed
@jcouv jcouv deleted the dev/jcouv/runtime-async branch November 5, 2025 03:39
@github-actions github-actions bot locked and limited conversation to collaborators Dec 5, 2025
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.

4 participants