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

uc-crux-llvm: User-provided specifications for functions #1003

Merged
merged 40 commits into from
Aug 2, 2022

Conversation

langston-barrett
Copy link
Contributor

@langston-barrett langston-barrett commented Jun 17, 2022

Fixes #982. Based on #1001. Best place to start reviewing would probably be by reading the user-facing docs (rendered).

@langston-barrett
Copy link
Contributor Author

Rebased onto #1001.

uc-crux-llvm/doc/specs.md Outdated Show resolved Hide resolved
uc-crux-llvm/doc/specs.md Outdated Show resolved Hide resolved
uc-crux-llvm/doc/specs.md Show resolved Hide resolved
uc-crux-llvm/src/UCCrux/LLVM/Main/Config/FromEnv.hs Outdated Show resolved Hide resolved
uc-crux-llvm/src/UCCrux/LLVM/Main/Config/Type.hs Outdated Show resolved Hide resolved
uc-crux-llvm/src/UCCrux/LLVM/View/FullType.hs Show resolved Hide resolved
uc-crux-llvm/src/UCCrux/LLVM/View/Postcond.hs Show resolved Hide resolved
uc-crux-llvm/src/UCCrux/LLVM/View/Precond.hs Show resolved Hide resolved
uc-crux-llvm/src/UCCrux/LLVM/View/Shape.hs Show resolved Hide resolved
uc-crux-llvm/src/UCCrux/LLVM/View/Specs.hs Show resolved Hide resolved
@langston-barrett
Copy link
Contributor Author

CI failure seems unrelated:

Run cd dependencies/mir-json && cargo install --locked --force
  cd dependencies/mir-json && cargo install --locked --force
  shell: /usr/bin/bash --noprofile --norc -e -o pipefail {0}
  env:
    CACHE_VERSION: 4
    NIX_PATH: nixpkgs=channel:[2](https://github.com/GaloisInc/crucible/runs/7522835904?check_suite_focus=true#step:10:2)1.11
    NS: nix shell github:nixos/nixpkgs/nixos-22.0[5](https://github.com/GaloisInc/crucible/runs/7522835904?check_suite_focus=true#step:10:5)#cabal-install github:nixos/nixpkgs/nixos-22.05#haskell.compiler.ghc902 nixpkgs#gmp nixpkgs#zlib nixpkgs#zlib.dev
warning: Using `cargo install` to install the binaries for the package in current working directory is deprecated, use `cargo install --path .` instead. Use `cargo build` if you want to simply build the package.
  Installing mir-json v0.1.0 (/home/runner/work/crucible/crucible/dependencies/mir-json)
    Updating crates.io index
error: failed to get `cargo` as a dependency of package `mir-json v0.1.0 (/home/runner/work/crucible/crucible/dependencies/mir-json)`

Caused by:
  failed to fetch `https://github.com/rust-lang/crates.io-index`

Caused by:
  error reading from the zlib stream; class=Zlib (5)
Error: Process completed with exit code [10](https://github.com/GaloisInc/crucible/runs/7522835904?check_suite_focus=true#step:10:11)1.

@langston-barrett
Copy link
Contributor Author

CI seems to be failing reliably at this step, though why that should be so for the Ubuntu 20.04/GHC 9.0.2 build and not the other Ubuntu 20.04 builds is a bit mystifying considering that this step doesn't involve GHC/Cabal.

Copy link
Contributor

@robdockins robdockins left a comment

Choose a reason for hiding this comment

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

I don't really have context for the UC-Crux changes, but the change to crucible proper seem fine to me.

These test the semantics of the nondetBranches construct.
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.

uc-crux-llvm: User-provided specifications for external functions
3 participants