-
Notifications
You must be signed in to change notification settings - Fork 4.8k
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: Delete IL verification failure helpers in importer #108996
Conversation
cc @dotnet/jit-contrib -- are there any CI legs worth running for this? |
We might want to do some renaming, say
I don't think so; no diffs should be sufficient. Our SPMI collection set is a bit impoverished right now, so maybe wait until that's fixed and rerun? |
Sure; I'll get the renaming out of the way in the meantime. |
Not sure when this will be resolved, so I thought I'd push my revisions now for review. |
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, this looks good.
/ba-g CLR_Tools_Tests pipeline is timing out across PRs. |
Part of #32648. Per this comment, I suspect this is the next (last?) step in removing IL verifier from the importer.