Skip to content

Disable StringBuilder marshalling tests on native AOT - #123530

Merged
MichalStrehovsky merged 1 commit into
mainfrom
MichalStrehovsky-patch-2
Jan 23, 2026
Merged

Disable StringBuilder marshalling tests on native AOT#123530
MichalStrehovsky merged 1 commit into
mainfrom
MichalStrehovsky-patch-2

Conversation

@MichalStrehovsky

Copy link
Copy Markdown
Member

Test started running with #123112

Cc @dotnet/ilc-contrib

CopilotAI review requested due to automatic review settings January 23, 2026 06:54
@github-actionsgithub-actionsBot added the needs-area-label An area label is needed to ensure this gets routed to the appropriate area owners label Jan 23, 2026
@MichalStrehovsky

Copy link
Copy Markdown
MemberAuthor

/azp run runtime-nativeaot-outerloop

@azure-pipelines

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

CopilotAI 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 disables three StringBuilder marshalling tests (ByValue, ByRef, and ReversePInvoke) on NativeAOT due to failures tracked in issue #123529. The tests started running after PR #123112 restored correct outerloop priority tracking and test execution.

Changes:

  • Added ActiveIssue attributes to three test methods to skip them on NativeAOT

@MichalStrehovsky

Copy link
Copy Markdown
MemberAuthor

Cool, these are not failing anymore and we got fresh new crashes.

@MichalStrehovsky

Copy link
Copy Markdown
MemberAuthor

/ba-g native AOT outerloop is known to be broken

@MichalStrehovsky
MichalStrehovsky merged commit c1b9b97 into mainJan 23, 2026
94 of 112 checks passed
@MichalStrehovsky
MichalStrehovsky deleted the MichalStrehovsky-patch-2 branch January 23, 2026 21:43
CopilotAI pushed a commit that referenced this pull request Jan 23, 2026
Test started running with #123112
Cc @dotnet/ilc-contrib
@github-actionsgithub-actionsBot locked and limited conversation to collaborators Feb 23, 2026
Sign up for freeto subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

needs-area-labelAn area label is needed to ensure this gets routed to the appropriate area owners

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants

@MichalStrehovsky@jkotas