Check for CodeSizeZero error before any assumptions about code existing

[masonwheeler]

Fixes #63227.

[ghost]

Tagging subscribers to this area: @JulieLeeMSFT
See info in area-owners.md if you want to be subscribed.

Issue Details

---

Fixes #63227.

Author:	masonwheeler
Assignees:	-
Labels:	area-ILVerification
Milestone:	-

[MichalStrehovsky]

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.

[JulieLeeMSFT]

@MichalStrehovsky if you reviewed the code, could you approve the code to merge it? Thanks.

[MichalStrehovsky]

@MichalStrehovsky if you reviewed the code, could you approve the code to merge it? Thanks.

This fix works around a bug in FindBasicBlocks. I would prefer FindBasicBlocks to be fixed instead because we have multiple users of that code. @masonwheeler would you be willing to change it to that instead?

[ghost]

This pull request has been automatically marked no-recent-activity because it has not had any activity for 14 days. It will be closed if no further activity occurs within 14 more days. Any new comment (by anyone, not necessarily the author) will remove no-recent-activity.

[ghost]

This pull request will now be closed since it had been marked no-recent-activity but received no further activity in the past 14 days. It is still possible to reopen or comment on the pull request, but please note that it will be locked if it remains inactive for another 30 days.