Skip to content

Align JIT HALT implementation with Interpreter and Verifier - #175

Merged
dvander merged 1 commit into
masterfrom
jit-halt
Feb 4, 2018
Merged

dvander merged 1 commit into
masterfrom
jit-halt

Conversation

@asherkin

@asherkin asherkin commented Feb 4, 2018

Copy link
Copy Markdown
Member

Next step is removing the code paths that can emit reachable halt's from spcomp (exit, sleep, and assert), but this fixes the crash for now.

For shell use it might be nice to have these work (and just map halt through to SetFailState for SM), but that is a discussion for the future.

Fixes #174

Next step is removing the code paths that can emit reachable `halt`'s from spcomp (exit, sleep, and assert), but this fixes the crash for now.
For shell use it might be nice to have these work (and just map `halt` through to `SetFailState` for SM), but that is a discussion for the future.

Fixes #174
@asherkin
asherkin requested a review from dvander February 4, 2018 12:47
@dvander

dvander commented Feb 4, 2018

Copy link
Copy Markdown
Member

How was this emitting when the verifier rejects it?

@asherkin

asherkin commented Feb 4, 2018

Copy link
Copy Markdown
Member Author

The SP version in SM 1.8-dev doesn't have #105.

@dvander

dvander commented Feb 4, 2018

Copy link
Copy Markdown
Member

Hah, okay.

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.

Sleep and Exit

2 participants