-
Notifications
You must be signed in to change notification settings - Fork 5.1k
JIT/EE interface cleanup #32521
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
JIT/EE interface cleanup #32521
Conversation
Delete methods on JIT/EE interface that are no longer used in CoreCLR
cc @dotnet/jit-contrib |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Would be good to verify this is zero-diff.
Seems like we could get rid of GTF_INX_REFARR_LAYOUT
entirely.
Superpmi not happy. |
The SuperPMI unit test finally found what it was designed to find. Nice. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks, Jan.
BOOL tiVerificationNeeded; | ||
// CoreCLR does not ever run IL verification. Compile out the verifier from the JIT by making this a constant. | ||
// TODO: Delete the verifier from the JIT? | ||
// BOOL tiVerificationNeeded; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Yeah, I think we should do this, but it doesn't have to be done now.
This reverts commit c73f640.
I have reverted the Example of asm diff: Baseline:
With
My guess is that that there is a place in register allocator (have not traced down where exactly) that compares all flags without interpreting them. |
|
Known issue: #32638 |
Delete methods on JIT/EE interface that are no longer used in CoreCLR