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

Properly match points-to and return value assertions during x86 verification #725

Merged
merged 9 commits into from
Jun 19, 2020

Conversation

chameco
Copy link
Contributor

@chameco chameco commented May 25, 2020

No description provided.

@chameco chameco marked this pull request as ready for review May 26, 2020 17:51
Copy link
Contributor

@andreistefanescu andreistefanescu left a comment

Choose a reason for hiding this comment

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

Looks good overall, let's add a couple of tests :-)

src/SAWScript/Crucible/LLVM/X86.hs Outdated Show resolved Hide resolved
@@ -632,31 +681,21 @@ assertPointsTo ::
Map MS.AllocIndex LLVMAllocSpec {- ^ Associates each AllocIndex with its specification -} ->
Map MS.AllocIndex C.LLVM.Ident {- ^ Associates each AllocIndex with its name -} ->
LLVMPointsTo LLVMArch {- ^ crucible_points_to statement from the precondition -} ->
X86Sim ()
X86Sim (LLVMOverrideMatcher md ())
Copy link
Contributor

Choose a reason for hiding this comment

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

monad in a monad :-)

src/SAWScript/Crucible/LLVM/X86.hs Show resolved Hide resolved
src/SAWScript/Crucible/LLVM/X86.hs Show resolved Hide resolved
src/SAWScript/Crucible/LLVM/X86.hs Outdated Show resolved Hide resolved
src/SAWScript/Crucible/LLVM/X86.hs Show resolved Hide resolved
src/SAWScript/Crucible/LLVM/X86.hs Outdated Show resolved Hide resolved
src/SAWScript/Crucible/LLVM/X86.hs Outdated Show resolved Hide resolved
Copy link
Contributor

@andreistefanescu andreistefanescu left a comment

Choose a reason for hiding this comment

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

👍

@chameco chameco merged commit 8989db7 into master Jun 19, 2020
@RyanGlScott RyanGlScott added the subsystem: x86 Issues related to verifying x86 binaries via Macaw label Dec 6, 2021
@RyanGlScott RyanGlScott deleted the x86-postconditions branch March 22, 2024 14:54
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
subsystem: x86 Issues related to verifying x86 binaries via Macaw
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants