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

Fix recently failures because upstream P4C changes to cstrings and annotations. #41

Merged
merged 1 commit into from
Nov 9, 2024

Conversation

asl
Copy link
Contributor

@asl asl commented Nov 9, 2024

No description provided.

@asl
Copy link
Contributor Author

asl commented Nov 9, 2024

@fruffy Will you please take a look?

@asl
Copy link
Contributor Author

asl commented Nov 9, 2024

The logic around https://github.com/p4gauntlet/toz3/pull/41/files#diff-dc7ce1551563fd6a0946f07c520318f7ec0e962400a9d17dc2b5c3413c33c44dR38 looks wrong to me, but I do not know the code to do proper fix :)

@fruffy
Copy link
Contributor

fruffy commented Nov 9, 2024

Thanks for fixing these. I had started on this but didn't have the time to finish yet.

The failure in the pruner is because of the changes to RemoveUnusedDeclarations. I haven't debugged that yet.

@fruffy fruffy changed the title Unbreak due to recent upstream changes Fix recently failures because upstream P4C changes to cstrings and annotations. Nov 9, 2024
@fruffy fruffy merged commit 0fe7c9e into p4gauntlet:master Nov 9, 2024
3 of 4 checks passed
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.

2 participants