Skip to content
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

Tighten PE verification in CompileAndVerify #23132

Closed
jcouv opened this issue Nov 11, 2017 · 1 comment
Closed

Tighten PE verification in CompileAndVerify #23132

jcouv opened this issue Nov 11, 2017 · 1 comment
Assignees
Labels
Area-Compilers Resolution-Fixed The bug has been fixed and/or the requested behavior has been implemented
Milestone

Comments

@jcouv
Copy link
Member

jcouv commented Nov 11, 2017

Currently, we either run the verification or we don't.
Instead, we should always run the verification and either expect that it succeeds or expect that it fails.

FYI @VSadov

@jcouv
Copy link
Member Author

jcouv commented Nov 14, 2017

Fixed

@jcouv jcouv closed this as completed Nov 14, 2017
@jcouv jcouv added the Resolution-Fixed The bug has been fixed and/or the requested behavior has been implemented label Nov 14, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Area-Compilers Resolution-Fixed The bug has been fixed and/or the requested behavior has been implemented
Projects
None yet
Development

No branches or pull requests

1 participant