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

crux-mir doesn't respect --assert-false-on-error #1080

Open
RyanGlScott opened this issue Apr 17, 2023 · 0 comments
Open

crux-mir doesn't respect --assert-false-on-error #1080

RyanGlScott opened this issue Apr 17, 2023 · 0 comments
Labels
bug crux MIR Issues relating to Rust/MIR support

Comments

@RyanGlScott
Copy link
Contributor

crux-mir provides an --assert-false-on-error option:

$ cabal run exe:crux-mir -- --help
...
[Crux]                --assert-false-on-error       when translation fails, assert false in output and keep going

However, the implementation of crux-mir never actually does anything with this option:

--let ?assertFalseOnError = assertFalse mirOpts
let ?assertFalseOnError = True

(This change was introduced in f9d4f8c.) Similarly, the test suite hard-codes the value here:

let ?assertFalseOnError = True

If we are going through the effort of passing ?assertFalseOnError as an implicit parameter, we ought to make it properly configurable by the user.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug crux MIR Issues relating to Rust/MIR support
Projects
None yet
Development

No branches or pull requests

1 participant