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

Change S.R.M.TypeName to be consumed from System.Reflection.Metadata package #101666

Merged
merged 2 commits into from Apr 29, 2024

Conversation

jkotas
Copy link
Member

@jkotas jkotas commented Apr 28, 2024

Contributes to #101541

Fixes #101628

@dotnet-issue-labeler dotnet-issue-labeler bot added the area-Tools-ILVerification Issues related to ilverify tool and IL verification in general label Apr 28, 2024
@jkotas jkotas added area-System.Reflection.Metadata and removed area-Tools-ILVerification Issues related to ilverify tool and IL verification in general labels Apr 28, 2024
Copy link
Contributor

Tagging subscribers to this area: @dotnet/area-system-reflection-metadata
See info in area-owners.md if you want to be subscribed.

@jkotas jkotas changed the title Change S.R.M.TypeName to be visible from System.Reflection.Metadata only Change S.R.M.TypeName to consumed from System.Reflection.Metadata package Apr 28, 2024
@jkotas
Copy link
Member Author

jkotas commented Apr 28, 2024

This should be enough to fix the build breaks in outer loop and source build. Further cleanup is tracked by #101541.

@jkotas
Copy link
Member Author

jkotas commented Apr 28, 2024

/azp run runtime-coreclr outerloop

Copy link

Azure Pipelines successfully started running 1 pipeline(s).

@jkotas jkotas changed the title Change S.R.M.TypeName to consumed from System.Reflection.Metadata package Change S.R.M.TypeName to be consumed from System.Reflection.Metadata package Apr 28, 2024
@jkotas
Copy link
Member Author

jkotas commented Apr 28, 2024

/azp run runtime-coreclr outerloop

Copy link

Azure Pipelines successfully started running 1 pipeline(s).

@jkotas
Copy link
Member Author

jkotas commented Apr 28, 2024

/azp run runtime-coreclr outerloop

Copy link

Azure Pipelines successfully started running 1 pipeline(s).

…package

Contributes to dotnet#101541

Fixes dotnet#101628

Better unreachable code elimination in CoreLib
@jkotas
Copy link
Member Author

jkotas commented Apr 29, 2024

/azp run runtime-coreclr outerloop

Copy link

Azure Pipelines successfully started running 1 pipeline(s).

Copy link
Member

@jeffhandley jeffhandley left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thank you, @jkotas. I appreciate you digging into this before I was able to get to it and before code flow was adversely disrupted.

@jkotas jkotas deleted the typenameparser branch April 29, 2024 07:13
matouskozak pushed a commit to matouskozak/runtime that referenced this pull request Apr 30, 2024
@adamsitnik
Copy link
Member

@jkotas @jeffhandley big thanks on fixing the issue when I was offline!

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.

Outerloop test builds broken
3 participants