-
Notifications
You must be signed in to change notification settings - Fork 152
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
[K-Improvement] Create Ordering
instance for Sentence
's subclasses
#3530
Comments
We need to find which subtypes of sentences need to implement the ordering like we already have for Rules and Productions. We also need a way to order all sentences between the types. Ex: Order -> Production Also, we would like to have a lazy val within this types sorted like in sortedProdutions |
Unit tests should help here. |
This was reverted because it may have been the cause of the release failing. |
Try to investigate by comparing We're putting the search results in a standard unordered set. Ask Freeman for access to a Mac machine. |
This Search calls
|
Had a deeper look, my above comment can be ignored. In the error case, the bad output is coming from |
For runtimeverification/k#3530 This fixes a couple of issues where the way kprint was sorting AC symbols involved spurious whitespaces and a problematic ordering of operations.
Motivation
We may need in the future to sort all sentences from a module to test a property or apply a transformation.
Following the last comment of #3523.
The text was updated successfully, but these errors were encountered: