Skip to content

Quantifiers#9

Open
daneshvar-amrollahi wants to merge 97 commits into
masterfrom
quantified
Open

Quantifiers#9
daneshvar-amrollahi wants to merge 97 commits into
masterfrom
quantified

Conversation

@daneshvar-amrollahi

@daneshvar-amrollahi daneshvar-amrollahi commented Sep 8, 2022

Copy link
Copy Markdown
Collaborator
  • Utilized Z3's support for quantified formulas
  • Summarized 9 string functions from uClibc using quantifiers in SpecialFunctionHandlers
  • Added summaries using quantifiers in Z3's Python API
  • Benchmarks are in klee/examples/quantified/ directory - enter make verify to run them.
  • Added README's

daneshvar and others added 30 commits June 17, 2022 17:34

@rishabh246 rishabh246 left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Looks good overall. Not surprising, since we've been through many iterations at this point.

Minor issues here and there, please fix.

One major issue is the lack of a README --- ideally there should be one both in the C examples and the python ones. But you can get to that after the presentation at the lab meeting.

Comment thread include/klee/Expr.h Outdated
Comment thread include/klee/Expr.h Outdated
Comment thread include/klee/Expr.h Outdated
Comment thread include/klee/klee.h Outdated
Comment thread include/klee/klee.h Outdated
Comment thread lib/Solver/Z3Builder.h Outdated
Comment thread lib/Solver/Z3Builder.h Outdated
Comment thread lib/Solver/Z3Solver.cpp Outdated
Comment thread lib/Solver/Z3Solver.cpp Outdated
Comment thread examples/quantified/strncmp/strncmp.c Outdated
@rishabh246

Copy link
Copy Markdown

@daneshvar-amrollahi the README for the C tests looks good. Can you add one for the python ones?

@daneshvar-amrollahi

Copy link
Copy Markdown
Collaborator Author

@daneshvar-amrollahi the README for the C tests looks good. Can you add one for the python ones?

Added.

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