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

Mem optimize #118

Merged
merged 1 commit into from
Oct 4, 2023
Merged

Mem optimize #118

merged 1 commit into from
Oct 4, 2023

Conversation

ladisgin
Copy link
Member

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.

@ladisgin ladisgin force-pushed the ladisgin/mem_optimize branch from 7447728 to ef4d1a4 Compare August 29, 2023 16:11
@codecov-commenter
Copy link

codecov-commenter commented Aug 29, 2023

Codecov Report

Merging #118 (f59aabb) into main (f5f7574) will decrease coverage by 7.39%.
The diff coverage is 81.79%.

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

@@            Coverage Diff             @@
##             main     #118      +/-   ##
==========================================
- Coverage   66.45%   59.06%   -7.39%     
==========================================
  Files         215      209       -6     
  Lines       29646    27348    -2298     
  Branches     6573     6167     -406     
==========================================
- Hits        19700    16152    -3548     
- Misses       7431     8874    +1443     
+ Partials     2515     2322     -193     
Files Coverage Δ
include/klee/Core/Interpreter.h 100.00% <ø> (ø)
include/klee/Module/KInstruction.h 100.00% <100.00%> (ø)
include/klee/Module/KModule.h 75.00% <100.00%> (-4.32%) ⬇️
lib/Basic/Statistics.cpp 76.31% <ø> (ø)
lib/Core/DistanceCalculator.cpp 87.23% <ø> (-5.08%) ⬇️
lib/Core/ExecutionState.h 93.42% <100.00%> (+0.08%) ⬆️
lib/Core/Executor.h 76.92% <100.00%> (ø)
lib/Core/Memory.h 77.19% <100.00%> (-10.53%) ⬇️
lib/Core/SpecialFunctionHandler.cpp 58.67% <100.00%> (-15.07%) ⬇️
lib/Core/TargetedExecutionManager.h 100.00% <ø> (ø)
... and 16 more

... and 60 files with indirect coverage changes

@ladisgin ladisgin force-pushed the ladisgin/mem_optimize branch 2 times, most recently from 1e9661e to c8ab5f6 Compare August 30, 2023 13:41
@ladisgin ladisgin changed the title Ladisgin/mem optimize Mem optimize Sep 12, 2023
@ladisgin ladisgin force-pushed the ladisgin/mem_optimize branch 7 times, most recently from 4ad1444 to 89df6a4 Compare October 2, 2023 11:28
@ladisgin ladisgin marked this pull request as ready for review October 2, 2023 13:16
include/klee/Module/KInstruction.h Outdated Show resolved Hide resolved
include/klee/Module/KModule.h Outdated Show resolved Hide resolved
lib/Module/KInstruction.cpp Show resolved Hide resolved
lib/Module/KInstruction.cpp Outdated Show resolved Hide resolved
lib/Module/KModule.cpp Outdated Show resolved Hide resolved
@ladisgin ladisgin force-pushed the ladisgin/mem_optimize branch from 2f4a8b8 to c5c898f Compare October 4, 2023 07:40
@ladisgin ladisgin requested a review from misonijnik October 4, 2023 08:41
@misonijnik misonijnik force-pushed the ladisgin/mem_optimize branch from c5c898f to 62d5357 Compare October 4, 2023 09:43
@misonijnik misonijnik merged commit 37f0da6 into main Oct 4, 2023
20 checks passed
@ladisgin ladisgin deleted the ladisgin/mem_optimize branch October 17, 2023 11:22
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