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

KCFG: Allow pretty printing #2643

Merged
merged 18 commits into from
Jun 7, 2022
Merged

KCFG: Allow pretty printing #2643

merged 18 commits into from
Jun 7, 2022

Conversation

nishantjr
Copy link
Contributor

No description provided.

nishantjr added 7 commits June 6, 2022 14:17
…alse positives.

We don't care if a node can loop back to itself somehow, but only that it
completes a loop by reaching some previous instance of itself in the history displayed so far.
We don't add pretty print as a member to Subst since this causes a circular
dependency between kprint.py and kast.py
The expected output looks ugly, but when we're using a real KPrint it should be fine.
@nishantjr nishantjr changed the title Kcfg pretty print KCFG: Allow pretty printing Jun 6, 2022
@nishantjr nishantjr requested a review from ehildenb June 6, 2022 20:26
@ehildenb
Copy link
Member

ehildenb commented Jun 6, 2022

Nice, this is great, also nice test :)

@nishantjr nishantjr self-assigned this Jun 7, 2022
@rv-jenkins rv-jenkins merged commit 35d68a4 into master Jun 7, 2022
@rv-jenkins rv-jenkins deleted the kcfg-pretty-print branch June 7, 2022 03:45
h0nzZik pushed a commit to h0nzZik/k that referenced this pull request Nov 24, 2022
…untimeverification#2113)

* haskell-backend/src/main/native/haskell-backend: 29888c356 - Update tests: change reviewer (#2777)

* haskell-backend/src/main/native/haskell-backend: 0521b0ea0 - Warn when RHS of a claim is \\bottom. (runtimeverification#2776)

* haskell-backend/src/main/native/haskell-backend: eefb9b41 - Continuation of Unification Loop Refactor (runtimeverification#2643)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants