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

WIP ipc verification #21

Draft
wants to merge 3 commits into
base: master
Choose a base branch
from
Draft

WIP ipc verification #21

wants to merge 3 commits into from

Conversation

priyasiddharth
Copy link
Collaborator

No description provided.

remove verbose vacuity output since it is unreliable (due to jump threading).
Instead print summary of actual passing/fail assertions and failing vacuity checks at the end
This is a libc interface that delegates to SEAHORN functions. The idea is that this libc can be dropped into a CMake project to enable quick startup
@priyasiddharth
Copy link
Collaborator Author

@agurfinkel This is a WIP (demo) of the ipc test. I have added a README.md with running time. The ipc_session_harness.c file contains documentation on what the test is doing.

Copy link
Collaborator

@agurfinkel agurfinkel left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

would be good to get a live explanation as well


int8_t *g_ptr0 = NULL;
size_t g_ptr0_size;
static int8_t *g_ptr0 = NULL;
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

these vars are not needed. but if they are used then definitely not static

VACUITY_CHECK_RE = r'^(?P<stream>Info|Error).*(?P<what>vacuity).*(?P<result>passed|failed).*sat\) (?P<debuginfo>.*)$'

def check_vacuity(line, passed_set, failed_set):
VACUITY_CHECK_RE = r'^(?P<stream>Info|Error).*(?P<what>vacuity|assertion).*(?P<result>passed|failed).*sat\) (?P<debuginfo>.*)$'
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

changes to this file are unrelated to pr

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