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

Debugger merging #86

Merged
merged 157 commits into from
Jan 8, 2022
Merged

Debugger merging #86

merged 157 commits into from
Jan 8, 2022

Conversation

giltho
Copy link
Contributor

@giltho giltho commented Oct 20, 2021

This is going to need a some of polishing first

A few notes, before or after merging this, one could:

  • improve the different required signatures once and for all.
    For example, yes the GInterpreter needs some State.S, and the lifter needs more.
    We could split the State.S into ExecutableState.S and LiftableState.S for example, or whatever equivalent that sounds right.
    I would, at the same time, take care of the differences between Symbolic and Concrete states, to clear off all the "Failwith "does not exist for concrete states"
  • Squash Wisl.CodeLoc with Gillian.Locations
  • If we give-up the idea of having multi-threaded logged execution (which seems entirely faire), we don't need UUIDMs, and that could increase the speed of generating the reports

matthew-ho added 30 commits May 26, 2021 11:50
@giltho giltho mentioned this pull request Oct 20, 2021
54 tasks
giltho added a commit that referenced this pull request Dec 20, 2021
giltho added a commit that referenced this pull request Dec 20, 2021
* Some bits of the #86 PR

* remove useless comment
@giltho giltho marked this pull request as ready for review January 8, 2022 17:57
@giltho giltho merged commit d4acb21 into master Jan 8, 2022
@NatKarmios NatKarmios deleted the debugger-merging branch November 17, 2022 15:43
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