Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Synthesize assembly names by default for ilproj-based tests #108997

Draft
wants to merge 4 commits into
base: main
Choose a base branch
from

Conversation

kg
Copy link
Member

@kg kg commented Oct 17, 2024

This reduces the number of test failures I get locally on a full run from 73 to 10, which should get the crossgen outerloop closer to green.

Background: We have a huge number of tests in .il form and many of them have .assembly directives with incorrect assembly names. In many cases this isn't a problem, but for crossgen it makes it impossible to locate the assembly pointed to by a reference because the filename won't match.

@dotnet-issue-labeler dotnet-issue-labeler bot added the needs-area-label An area label is needed to ensure this gets routed to the appropriate area owners label Oct 17, 2024
@kg
Copy link
Member Author

kg commented Oct 17, 2024

/azp run runtime-coreclr crossgen2 outerloop

Copy link

Azure Pipelines successfully started running 1 pipeline(s).

@kg
Copy link
Member Author

kg commented Oct 18, 2024

/azp run runtime-coreclr crossgen2 outerloop

Copy link

Azure Pipelines successfully started running 1 pipeline(s).

@lambdageek lambdageek added test-enhancement Improvements of test source code area-ILTools-coreclr labels Oct 18, 2024
Copy link
Contributor

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

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area-ILTools-coreclr needs-area-label An area label is needed to ensure this gets routed to the appropriate area owners test-enhancement Improvements of test source code
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants