-
Notifications
You must be signed in to change notification settings - Fork 5.3k
Check for CodeSizeZero error before any assumptions about code existing #63239
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
Conversation
|
Tagging subscribers to this area: @JulieLeeMSFT Issue DetailsFixes #63227.
|
|
The reason I suggested putting this check into FindBasicBlocks is because IlImporter.cs is shared with multiple projects and this would make sure they all handle it instead of throwing the boneheaded exception. This fix works for IlVerify but the benefit is limited. |
|
@MichalStrehovsky if you reviewed the code, could you approve the code to merge it? Thanks. |
This fix works around a bug in |
|
This pull request has been automatically marked |
|
This pull request will now be closed since it had been marked |
Fixes #63227.