Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Enable marking if commands as {:allow_split} or not #960

Closed

Conversation

keyboardDrummer
Copy link
Collaborator

@keyboardDrummer keyboardDrummer commented Oct 4, 2024

Review the commits separately. Refactorings have been placed in the first commit shown in the commits tab.

Changes

  • Fix concurrency issue that could create a crash when using multiple cores and simple programs (commit)
  • Introduce the attribute {:allow_split} that can be placed on IfCmd and GotoCmd. When using {:isolate "paths"} only, jumps with {:allow_split} will trigger splitting of the VC.

Testing

Updated existing tests so they exercise the presence and absence of {:allow_split}

keyboardDrummer added a commit that referenced this pull request Oct 7, 2024
Extracted classes into separate files, corrected variable names, and
moved some methods to extension classes
fabiomadge added a commit to fabiomadge/boogie that referenced this pull request Oct 15, 2024
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant