-
Notifications
You must be signed in to change notification settings - Fork 22
Remove Formula
#326
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
Closed
Closed
Remove Formula
#326
Changes from 56 commits
Commits
Show all changes
57 commits
Select commit
Hold shift + click to select a range
163e9f3
Add `Asrt.t = Asrt.simple list`
N1ark 2a8672a
This compiles somehow
N1ark 45a6595
Fix C-gen for emp
N1ark 3ab57e2
Merge branch 'fix-fixes' into asrt-as-list
N1ark 1b60fb9
Simplify stuff
N1ark 5c0f6d1
Some cleaning
N1ark 6cd169d
GILLIAN C !!!!!!!!!! 🫵
N1ark 4364169
Gillian-C PLEASE
N1ark d8f2731
Remove extranuous emphs in gil gen
N1ark baa28c4
Cleanup
N1ark c6c72d0
Fix stack overflow !! (oops)
N1ark c6a7574
Rename `Asrt.simple` to `Asrt.atom`
N1ark 92caec0
Merge branch 'master' into asrt-simple-list
N1ark 8a4c09b
Fix CI
N1ark 2ccd90f
Fix incorrect `auto_unfold`
N1ark ac9dc4d
So this works but is not Good(tm)
N1ark efb515d
That's a fix i guess : (
N1ark 77b69e7
Re-sort assertions
N1ark c91d2b2
Re-sort !!
N1ark df0be14
Merge branch 'master' into asrt-simple-list
N1ark 9387638
Cleanup
N1ark c869ac8
Make recovering of consume fuel-based
N1ark f721536
Use pipes
N1ark a8d346f
Disable asrt sort in produce, avoids branch expl
N1ark 156e062
Nevermind I guess we're sorting
N1ark 93595ac
Remove from syntax
N1ark bafae51
Update FOLogic/
N1ark ad56f51
Update general_semantics/
N1ark a278b5d
Update symbolic_semantics/
N1ark 4368ddd
Update concrete_semantics/
N1ark 0aa7e79
Update BiAbduction
N1ark 8403b02
Update Abstraction
N1ark 59b874b
Update Monadic
N1ark e9eb5a2
Update smt / GIL_Syntax
N1ark 5657f80
Update Gillian-C
N1ark a829b95
Update Gillian-C2
N1ark 3e948ee
Update Gillian-JS
N1ark 0c2bb48
Update WISL
N1ark b39426c
Update ppx_sat
N1ark b29f6df
Update core
N1ark 9f47e0d
Simplify parser
N1ark f81d402
Small simplifications, fix `Expr.(> >= >. >=.)`
N1ark 3c0625c
Simplify parser, fix reductions
N1ark 8d15c7d
Appease CI?
N1ark cee19c4
Fix operator precedence
N1ark d5ee07c
Use Sacha's new `resolve_expr_to_location`
N1ark 0d921a4
Merge branch 'asrt-simple-list' into no-more-formula
N1ark 9b4f431
Collapse reductions (wow)
N1ark c027e8c
Fix reducing to true bc of PFS
N1ark 4518574
Fix reducing away all UnOps
N1ark 9a4c36a
Don't check PFS for Not
N1ark f83afb9
Disable other rule abt reducing to l-len
N1ark 641c12b
Small optim `Type_env.get_unsafe`
N1ark 6f812b1
Fix reducing before comparing with `Nono`
N1ark 3f536b3
Remove `reduce_formula` !!!!
N1ark 3834539
Remove comments + unneeded reductions
N1ark 8111475
Fix `forall` precedences (thanks @v-gb !)
N1ark File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
There are no files selected for viewing
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Oops, something went wrong.
Oops, something went wrong.
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Uh oh!
There was an error while loading. Please reload this page.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
(Hi, I wrote the ocamlformat formatting change for the hash operators that's probably the reason for this PR. I was just curious to see what you were doing in response).
I believe this change alters the formula, because the precedence in the initial source code is confusing. When I renamed these operators, it resulted in this instead (the link doesn't seem to navigate to the proper file for some reason? but you can search for
forall).(forall i. zero) <= i && ..vsforall i, (zero <= i && ..)IIUC. I made that diff by parsing the source file with ocamlformat, modifying the ast, and printing the ast back.Anyway, just a random remark to say it seems it'd reduce the risk of regression to incorporate my linked commit or something similar, before your further refactoring.
Uh oh!
There was an error while loading. Please reload this page.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Hi!
Thanks for checking it out :) Yes indeed I didn't notice, these expressions were wrong, thank you for pointing it out, just fixed it in 8111475. It was because the precedence of
#..operators is higher than that of function application so when updating the syntax I missed it. (Un)luckily these bits of code were never run so I didn't notice :pYeah the change is partly due by the ocamlformat change, though in general I think it makes more sense to define operators that have sensible precedence from the get go :)