Skip to content

Conversation

@kg
Copy link
Member

@kg kg commented Aug 15, 2025

Issue #108255 was fixed by #118594

@kg
Copy link
Member Author

kg commented Aug 15, 2025

/azp run runtime-coreclr crossgen2 outerloop

@azure-pipelines
Copy link

Azure Pipelines successfully started running 1 pipeline(s).

@kg
Copy link
Member Author

kg commented Aug 15, 2025

/azp run runtime-coreclr crossgen2 outerloop

@azure-pipelines
Copy link

Azure Pipelines successfully started running 1 pipeline(s).

@kg kg marked this pull request as ready for review August 15, 2025 18:15
@kg kg requested review from Copilot and mangod9 August 15, 2025 18:15
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 crossgen exclusions that were previously added to work around issue 108255. The change suggests that the underlying issue has been resolved, allowing these test paths to be included in crossgen2 testing again.

  • Removes 5 test exclusions from the crossgen exclusion list
  • Re-enables testing for various JIT regression tests and base services tests
Comments suppressed due to low confidence (1)

src/tests/issues.targets:543

  • Removing these test exclusions without verifying that the underlying issue 108255 has been fixed could lead to test failures. The PR should include evidence that these tests now pass with crossgen2 or reference the fix that resolved issue 108255.
        <ExcludeList Include="$(XunitTestBinBase)/JIT/Methodical/Arrays/misc/arrres_il_r/**">

@kg kg merged commit 061acba into dotnet:main Aug 15, 2025
121 of 123 checks passed
@github-actions github-actions bot locked and limited conversation to collaborators Sep 15, 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.

2 participants