Skip to content

Conversation

@MichalStrehovsky
Copy link
Member

I'm reading through the runtime async PR. This caught my interest, found out it's unused.

I'm reading through the runtime async PR. This caught my interest, found out it's unused.
Copilot AI review requested due to automatic review settings October 31, 2025 13:05
@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 Oct 31, 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 removes the unused IsStructMethodOperatingOnCopy() method from the MethodDesc class in CoreCLR's method implementation.

  • Removes dead code: The IsStructMethodOperatingOnCopy() method which was checking for async variant implementations on value type instances

@am11 am11 added area-VM-coreclr and removed needs-area-label An area label is needed to ensure this gets routed to the appropriate area owners labels Oct 31, 2025
@dotnet-policy-service
Copy link
Contributor

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

@jkotas
Copy link
Member

jkotas commented Oct 31, 2025

/ba-g infrastructure timeouts

@jkotas jkotas merged commit 5ef9ed1 into dotnet:main Oct 31, 2025
102 of 108 checks passed
@MichalStrehovsky MichalStrehovsky deleted the unused branch October 31, 2025 21:23
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.

3 participants