-
Notifications
You must be signed in to change notification settings - Fork 152
CFG visualization and progress saving #1447
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
Merged
Merged
Changes from all commits
Commits
Show all changes
33 commits
Select commit
Hold shift + click to select a range
4086c60
kevm-pyk/__main__: _LOGGER.error => ValueError
ehildenb 9d99977
kevm-pyk/__main__: add foundry-show-cfg endpoint
ehildenb 5540670
bin/kevm: expose kevm foundry-show-cfg
ehildenb 307034c
kevm-pyk/__main__: add ability to display individual nodes too
ehildenb 6cd929f
kevm-pyk/kevm: bring in KEVM.add_language_invariants
ehildenb 97f0213
kevm-pyk/solc_to_k: make sure we are starting executions in the langu…
ehildenb 5f46ba9
kevm-pyk/__main__: foundry_out/kcfgs for cfgs, not in kompiled directory
ehildenb 832a945
kevm: remove debugging message
ehildenb 82c9fdb
kevm-pyk/__main__: add --node-delta option too foundry-show-cfg
ehildenb 1e909a6
kevm-pyk/utils: add KCFG__replace_node routine
ehildenb c51fa23
kevm-pyk/utils: bring back sanitize_config
ehildenb b04e372
kevm-pyk/__main__: option for simplifying initial state at startup
ehildenb ad4fe82
kevm-pyk/__main__: similify target node at startup too
ehildenb f86ed95
kevm-pyk/__main__: typo
ehildenb 95cffaf
kevm-pyk/__main__: correct how CFG passed/failed logic is handled
ehildenb cd53f85
bin/kevm, kevm-pyk/__main__: add foundry-list command for seeing KCFG…
ehildenb 0518354
bin/kevm, kevm-pyk/__main__: rename show-cfg => show
ehildenb 37ebfc3
kevm-pyk/__main__: better list message
ehildenb c496a22
kevm-pyk/__main__: remove --store-depth option
ehildenb daa4499
kevm-pyk/__main__: factor out _write_cfg method
ehildenb 17974f0
kevm-pyk/kevm: update KCFG with results of running prover
ehildenb d68760a
kevm-pyk/__main__: write cfg to disk
ehildenb 42ff567
kevm-pyk/__main__: explore from whichever is the fronteir on each call
ehildenb 9aaee3d
kevm-pyk/__main__: remove vestigial store_depth
ehildenb 37bfcbb
kevm-pyk/__main__: better name for specs
ehildenb 8f91400
Makefile: dont simplify init/target on CI
ehildenb 3d23575
kevm-pyk/__main__: more clear about status of proofs
ehildenb 7d5c5af
kevm-pyk/__main__: silly logic error
ehildenb 83fd11b
kevm-pyk/__main__: make sure to add expanded attribute to explored nodes
ehildenb de5d300
kevm-pyk/__main__: correct variable names
ehildenb 95671f9
Merge remote-tracking branch 'upstream/master' into save-progress-vis…
ehildenb 5a54cd5
kevm-pyk/__main__: make sure we sanitize configs back from prover
ehildenb 42d9203
Merge branch 'master' into save-progress-visualize
rv-jenkins 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.
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.
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.
Consider extracting
definition_dir
,use_dir
,kcfg_dir
,kcfg_file
as functions of a new moduleconfig.py
, e.g.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.
You mean as helpers so we're not re-computing them over and again? Maybe they could be fields in the
KEVM
class so they're jsut set at function init?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.
I took a quick look at this, and seems like more trouble than it's worth at the moment. Will consider it in my local tree, see if I can find a solution.
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.
Re-computing the values is fine. The point is to have a single definition of these expressions so that e.g. the folder layout can be easily reconfigured.