Skip to content

Delete some dead code`#91018

Merged
jkotas merged 1 commit into
dotnet:mainfrom
jkotas:class-must-be-restored
Aug 24, 2023
Merged

Delete some dead code`#91018
jkotas merged 1 commit into
dotnet:mainfrom
jkotas:class-must-be-restored

Conversation

@jkotas

@jkotas jkotas commented Aug 23, 2023

Copy link
Copy Markdown
Member

No description provided.

@ghost ghost added the area-VM-coreclr label Aug 23, 2023
@ghost ghost assigned jkotas Aug 23, 2023
@jkotas jkotas requested a review from VSadov August 23, 2023 20:35
@jkotas

jkotas commented Aug 23, 2023

Copy link
Copy Markdown
Member Author

Leftover from #89548

@VSadov

VSadov commented Aug 23, 2023

Copy link
Copy Markdown
Member

As I remember fClassMustBeRestored was removed in the big change (and that part was not causing any issues), but later was undone together with all JIT changes.

@jkotas

jkotas commented Aug 23, 2023

Copy link
Copy Markdown
Member Author

Right

VSadov
VSadov approved these changes Aug 23, 2023
View reviewed changes

@VSadov VSadov left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

LGTM!

@jkotas jkotas mentioned this pull request Aug 24, 2023
Closed
@jkotas jkotas merged commit b0d2596 into dotnet:main Aug 24, 2023
@jkotas jkotas deleted the class-must-be-restored branch August 24, 2023 00:26
@ghost ghost locked as resolved and limited conversation to collaborators Sep 23, 2023
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Reviewers

@VSadov VSadov VSadov approved these changes

Assignees

@jkotas jkotas

Labels

area-VM-coreclr

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@jkotas @VSadov