Skip to content

Conversation

@michael-schwarz
Copy link
Member

Any expression may be cast to void according to the standard if an expression is only evaluated for its side-effects. We currently limit this to (TInt _ | TEnum _ | TPtr _ ) for some reason.

Closes #91

@michael-schwarz
Copy link
Member Author

The build failures for old versions of OCaml are strange and probably unrelated as they cause all tests to fail not just the new ones.

Maybe due to old dune versions and some after effect of #104 @sim642?

Output for failing OCaml <4.07.1

File "test/dune", line 1, characters 0-180:
1 | (rule
2 |  (alias runtest)
3 |  (deps (alias_rec ../cilly) (package goblint-cil) (source_tree .))
4 |  (action (setenv "CC" "\"%{read-lines:../bin/real-gcc}\"" (run ./testcil -r --regrtest))))
(cd _build/default/test && ./testcil -r --regrtest)
Test infrastructure for CIL on linux
There are 431 tests enabled

Output for working OCaml

(cd _build/default/test && ./testcil -r --regrtest)
Test infrastructure for CIL on linux
There are 431 tests enabled

@michael-schwarz
Copy link
Member Author

These tests indeed already seem to fail on develop now.

@sim642
Copy link
Member

sim642 commented Jul 24, 2022

That's odd, because they all passed at the time and the first attempt (from two days ago) on develop also passed. Maybe something on opam of our dependencies regressed since then?

@sim642
Copy link
Member

sim642 commented Jul 25, 2022

Looks like a problem with the seq package update: c-cube/seq#14. We can just ignore those failures for the moment.

@michael-schwarz michael-schwarz merged commit 8eb9ff7 into develop Jul 26, 2022
@michael-schwarz michael-schwarz deleted the issue_91 branch July 26, 2022 07:33
@sim642 sim642 added this to the 2.0.0 milestone Aug 12, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Error for tgmath functions (if argument contains a pointer dereference?)

3 participants