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

Create alpha-equvalency format solver #135

Merged
merged 3 commits into from
Oct 23, 2023

Conversation

dim8art
Copy link

@dim8art dim8art commented Oct 16, 2023

Summary:

Checklist:

  • The PR addresses a single issue. If it can be divided into multiple independent PRs, please do so.
  • The PR is divided into a logical sequence of commits OR a single commit is sufficient.
  • There are no unnecessary commits (e.g. commits fixing issues in a previous commit in the same PR).
  • Each commit has a meaningful message documenting what it does.
  • All messages added to the codebase, all comments, as well as commit messages are spellchecked.
  • The code is commented OR not applicable/necessary.
  • The patch is formatted via clang-format OR not applicable (if explicitly overridden leave unchecked and explain).
  • There are test cases for the code you added or modified OR no such test cases are required.

@dim8art dim8art force-pushed the AlphaEquiv branch 6 times, most recently from 6451dda to 6bb74c6 Compare October 17, 2023 02:46
@codecov-commenter
Copy link

codecov-commenter commented Oct 17, 2023

Codecov Report

Merging #135 (1440409) into main (c3553c0) will increase coverage by 0.15%.
Report is 2 commits behind head on main.
The diff coverage is 85.35%.

❗ Current head 1440409 differs from pull request most recent head e920c44. Consider uploading reports for the commit e920c44 to get more accurate results

@@            Coverage Diff             @@
##             main     #135      +/-   ##
==========================================
+ Coverage   66.60%   66.76%   +0.15%     
==========================================
  Files         216      217       +1     
  Lines       29746    29927     +181     
  Branches     6573     6594      +21     
==========================================
+ Hits        19813    19980     +167     
+ Misses       7426     7423       -3     
- Partials     2507     2524      +17     
Files Coverage Δ
include/klee/ADT/SparseStorage.h 89.83% <100.00%> (ø)
include/klee/Expr/Parser/Parser.h 100.00% <ø> (ø)
include/klee/Solver/SolverUtil.h 63.79% <100.00%> (+2.62%) ⬆️
lib/Core/Executor.cpp 74.71% <100.00%> (ø)
lib/Expr/SourceBuilder.cpp 78.43% <100.00%> (+1.83%) ⬆️
lib/Solver/CachingSolver.cpp 73.91% <100.00%> (+2.17%) ⬆️
lib/Solver/ConstructSolverChain.cpp 91.30% <100.00%> (+0.39%) ⬆️
lib/Solver/SolverCmdLine.cpp 100.00% <ø> (ø)
tools/kleaver/main.cpp 74.89% <100.00%> (+0.10%) ⬆️
lib/Core/Memory.cpp 75.92% <66.66%> (ø)
... and 6 more

... and 10 files with indirect coverage changes

@dim8art dim8art force-pushed the AlphaEquiv branch 3 times, most recently from dff392b to 7f8c355 Compare October 18, 2023 14:05
@dim8art dim8art force-pushed the AlphaEquiv branch 3 times, most recently from b19b614 to 778eca3 Compare October 22, 2023 01:53
lib/Expr/AlphaBuilder.cpp Outdated Show resolved Hide resolved
lib/Expr/SourceBuilder.cpp Outdated Show resolved Hide resolved
@misonijnik misonijnik requested review from misonijnik and removed request for misonijnik October 23, 2023 15:08
@misonijnik misonijnik marked this pull request as ready for review October 23, 2023 15:09
@misonijnik misonijnik merged commit 001f9ef into UnitTestBot:main Oct 23, 2023
20 checks passed
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.

3 participants