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

Treat match_kind as a regular type #3099

Merged
merged 1 commit into from
Feb 27, 2022
Merged

Conversation

mihaibudiu
Copy link
Contributor

Signed-off-by: Mihai Budiu [email protected]

Fixes #3091

Copy link
Contributor

@jafingerhut jafingerhut left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM. As usual, I reviewed the new test cases and their midend outputs, and they look correct to me. I verified that without these changes to the compiler, the test program fails, and it passes with these compiler changes. Thanks!

Copy link
Contributor

@jnfoster jnfoster left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@jafingerhut jafingerhut merged commit 7988ea2 into p4lang:main Feb 27, 2022
@jafingerhut
Copy link
Contributor

Given 2 approvals and all tests passing, I will go ahead and merge this in. Feel free to tell me to revert it if I have moved too quickly here.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Questions on where match_kind values should be allowed
3 participants