Skip to content

Conversation

@MichalStrehovsky
Copy link
Member

This is likely no longer needed since the workaround code will kick in: #123378 (comment)

This is likely no longer needed since the workaround code will kick in: #123378 (comment)
Copilot AI review requested due to automatic review settings January 27, 2026 13:38
@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 Jan 27, 2026
@MichalStrehovsky
Copy link
Member Author

/azp run runtime-nativeaot-outerloop

@azure-pipelines
Copy link

Azure Pipelines successfully started running 1 pipeline(s).

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 a workaround that was disabling async tests on native AOT ARM64 architecture. The workaround was introduced to address issue #121871, which should now be resolved by PR #123378 that added allocMem support for allocating multiple data chunks.

Changes:

  • Remove the PropertyGroup block that disabled async test builds for native AOT ARM64

@jkotas jkotas added area-NativeAOT-coreclr runtime-async and removed needs-area-label An area label is needed to ensure this gets routed to the appropriate area owners labels Jan 27, 2026
@dotnet-policy-service
Copy link
Contributor

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

@jakobbotsch
Copy link
Member

This is likely no longer needed since the workaround code will kick in: #123378 (comment)

No, I broke it again to make #123378 have no functional changes. But I will make sure to fold this change into #123433

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.

4 participants