Skip to content

Cleanup of SMT2 solver test execution and CMake support [blocks: #4063] #4121

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

Merged
merged 4 commits into from
Feb 14, 2019
Merged
Show file tree
Hide file tree
Changes from all commits
Commits
File filter

Filter by extension

Filter by extension

Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
2 changes: 1 addition & 1 deletion .travis.yml
Original file line number Diff line number Diff line change
Expand Up @@ -304,7 +304,7 @@ script:
- if [ -e bin/gcc ] ; then export PATH=$PWD/bin:$PATH ; fi ;
- env UBSAN_OPTIONS=print_stacktrace=1 make -C regression test-parallel "CXX=${COMPILER} ${EXTRA_CXXFLAGS}" -j2 JOBS=2
- UBSAN_OPTIONS=print_stacktrace=1 make -C regression/cbmc test-paths-lifo
- scripts/delete_failing_smt2_solver_tests ; env PATH=$PATH:`pwd`/src/solvers UBSAN_OPTIONS=print_stacktrace=1 make -C regression/cbmc test-cprover-smt2
- env PATH=$PATH:`pwd`/src/solvers UBSAN_OPTIONS=print_stacktrace=1 make -C regression/cbmc test-cprover-smt2
- make -C unit "CXX=${COMPILER} ${EXTRA_CXXFLAGS}" -j2
- make -C unit test
- env UBSAN_OPTIONS=print_stacktrace=1 make -C jbmc/regression test-parallel "CXX=${COMPILER} ${EXTRA_CXXFLAGS}" -j2 JOBS=2
Expand Down
1 change: 0 additions & 1 deletion CODEOWNERS
Original file line number Diff line number Diff line change
Expand Up @@ -60,7 +60,6 @@
/jbmc/regression/

/scripts/ @diffblue/devops @thk123 @forejtv @peterschrammel
/scripts/delete_failing_smt2_solver_tests
/scripts/expected_doxygen_warnings.txt

/.travis.yml @diffblue/devops @thk123 @forejtv @peterschrammel @chrisr-diffblue
1 change: 0 additions & 1 deletion buildspec-linux-clang.yml
Original file line number Diff line number Diff line change
Expand Up @@ -22,7 +22,6 @@ phases:
commands:
- make -C unit test CXX='ccache /usr/bin/clang++-7' CXX_FLAGS='-Qunused-arguments'
- make -C regression test
- scripts/delete_failing_smt2_solver_tests
- env PATH=$PATH:`pwd`/src/solvers make -C regression/cbmc test-cprover-smt2
- make -C jbmc/unit test CXX='ccache /usr/bin/clang++-7' CXX_FLAGS='-Qunused-arguments'
- make -C jbmc/regression test
Expand Down
4 changes: 4 additions & 0 deletions buildspec-windows.yml
Original file line number Diff line number Diff line change
Expand Up @@ -40,6 +40,10 @@ phases:
$env:Path = "C:\tools\cygwin\bin;$env:Path"
cmd /c 'call "C:\Program Files (x86)\Microsoft Visual Studio 14.0\VC\vcvarsall.bat" x64 && bash -c "make -C regression test BUILD_ENV=MSVC" '

- |
$env:Path = "$pwd\src\solvers;C:\tools\cygwin\bin;$env:Path"
cmd /c 'call "C:\Program Files (x86)\Microsoft Visual Studio 14.0\VC\vcvarsall.bat" x64 && bash -c "make -C regression/cbmc test-cprover-smt2 BUILD_ENV=MSVC" '

- |
$env:Path = "C:\tools\cygwin\bin;$env:Path"
cmd /c 'call "C:\Program Files (x86)\Microsoft Visual Studio 14.0\VC\vcvarsall.bat" x64 && bash -c "make -C regression/cbmc test-paths-lifo BUILD_ENV=MSVC" '
Expand Down
1 change: 0 additions & 1 deletion buildspec.yml
Original file line number Diff line number Diff line change
Expand Up @@ -26,7 +26,6 @@ phases:
- make -C unit test
- make -C regression test
- make -C regression/cbmc test-paths-lifo
- scripts/delete_failing_smt2_solver_tests
- env PATH=$PATH:`pwd`/src/solvers make -C regression/cbmc test-cprover-smt2
- make -C jbmc/unit test
- make -C jbmc/regression test
Expand Down
2 changes: 1 addition & 1 deletion regression/cbmc/Anonymous_Struct3/test.desc
Original file line number Diff line number Diff line change
@@ -1,4 +1,4 @@
CORE
CORE broken-smt-backend
Copy link
Contributor

Choose a reason for hiding this comment

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

typo in commit message? "easier to repeated build" -> "easier to repeatedly build"?
"regression test" -> "regression tests"?

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

Thanks a lot!

main.c

^EXIT=0$
Expand Down
2 changes: 1 addition & 1 deletion regression/cbmc/Array_operations1/test.desc
Original file line number Diff line number Diff line change
@@ -1,4 +1,4 @@
CORE
CORE broken-smt-backend
main.c

^EXIT=10$
Expand Down
2 changes: 1 addition & 1 deletion regression/cbmc/Bitfields3/paths.desc
Original file line number Diff line number Diff line change
@@ -1,4 +1,4 @@
CORE
CORE broken-smt-backend
main.c
--pointer-check --bounds-check --paths lifo
^EXIT=0$
Expand Down
2 changes: 1 addition & 1 deletion regression/cbmc/Bitfields3/test.desc
Original file line number Diff line number Diff line change
@@ -1,4 +1,4 @@
CORE
CORE broken-smt-backend
main.c
--pointer-check --bounds-check
^EXIT=0$
Expand Down
12 changes: 12 additions & 0 deletions regression/cbmc/CMakeLists.txt
Original file line number Diff line number Diff line change
Expand Up @@ -8,3 +8,15 @@ add_test_pl_profile(
"-C;-X;thorough-paths;-X;smt-backend;-s;paths-lifo"
"CORE"
)

Copy link
Contributor

Choose a reason for hiding this comment

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

typo in commit message: "partity"

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

Thanks, fixed!

add_test_pl_profile(
"cbmc-cprover-smt2"
"$<TARGET_FILE:cbmc> --cprover-smt2"
"-C;-X;broken-smt-backend;-s;cprover-smt2"
"CORE"
)
set_property(
TEST "cbmc-cprover-smt2-CORE"
PROPERTY ENVIRONMENT
"PATH=$ENV{PATH}:${CMAKE_BINARY_DIR}/bin"
)
2 changes: 1 addition & 1 deletion regression/cbmc/Empty_struct1/test.desc
Original file line number Diff line number Diff line change
@@ -1,4 +1,4 @@
CORE
CORE broken-smt-backend
main.c

^EXIT=0$
Expand Down
2 changes: 1 addition & 1 deletion regression/cbmc/Endianness4/test.desc
Original file line number Diff line number Diff line change
@@ -1,4 +1,4 @@
CORE
CORE broken-smt-backend
main.c

^EXIT=0$
Expand Down
2 changes: 1 addition & 1 deletion regression/cbmc/Endianness6/test.desc
Original file line number Diff line number Diff line change
@@ -1,4 +1,4 @@
CORE
CORE broken-smt-backend
main.c
--big-endian
^EXIT=0$
Expand Down
2 changes: 1 addition & 1 deletion regression/cbmc/Float-div2/test.desc
Original file line number Diff line number Diff line change
@@ -1,4 +1,4 @@
CORE
CORE broken-smt-backend
main.c
--floatbv
^EXIT=0$
Expand Down
2 changes: 1 addition & 1 deletion regression/cbmc/Float-div3/test.desc
Original file line number Diff line number Diff line change
@@ -1,4 +1,4 @@
CORE
CORE broken-smt-backend
main.c
--floatbv
^EXIT=0$
Expand Down
2 changes: 1 addition & 1 deletion regression/cbmc/Float-no-simp1/test.desc
Original file line number Diff line number Diff line change
@@ -1,4 +1,4 @@
CORE
CORE broken-smt-backend
main.c
--no-simplify
^EXIT=0$
Expand Down
2 changes: 1 addition & 1 deletion regression/cbmc/Float-no-simp2/test.desc
Original file line number Diff line number Diff line change
@@ -1,4 +1,4 @@
CORE
CORE broken-smt-backend
main.c
--floatbv --no-simplify
^EXIT=0$
Expand Down
2 changes: 1 addition & 1 deletion regression/cbmc/Float-no-simp3/test.desc
Original file line number Diff line number Diff line change
@@ -1,4 +1,4 @@
CORE
CORE broken-smt-backend
main.c
--floatbv --no-simplify
^EXIT=0$
Expand Down
2 changes: 1 addition & 1 deletion regression/cbmc/Float-no-simp4/test.desc
Original file line number Diff line number Diff line change
@@ -1,4 +1,4 @@
CORE
CORE broken-smt-backend
main.c
--floatbv --no-simplify
^EXIT=0$
Expand Down
2 changes: 1 addition & 1 deletion regression/cbmc/Float-no-simp5/test.desc
Original file line number Diff line number Diff line change
@@ -1,4 +1,4 @@
CORE
CORE broken-smt-backend
main.c
--floatbv --no-simplify
^EXIT=0$
Expand Down
2 changes: 1 addition & 1 deletion regression/cbmc/Float-no-simp6/test.desc
Original file line number Diff line number Diff line change
@@ -1,4 +1,4 @@
CORE
CORE broken-smt-backend
main.c
--floatbv --no-simplify
^EXIT=0$
Expand Down
2 changes: 1 addition & 1 deletion regression/cbmc/Float-no-simp7/test.desc
Original file line number Diff line number Diff line change
@@ -1,4 +1,4 @@
CORE
CORE broken-smt-backend
main.c
--floatbv --no-simplify
^EXIT=0$
Expand Down
2 changes: 1 addition & 1 deletion regression/cbmc/Float-smt2-1/test.desc
Original file line number Diff line number Diff line change
@@ -1,4 +1,4 @@
CORE smt-backend
CORE smt-backend broken-smt-backend
main.c
--smt2
^EXIT=10$
Expand Down
2 changes: 1 addition & 1 deletion regression/cbmc/Float-to-double2/test.desc
Original file line number Diff line number Diff line change
@@ -1,4 +1,4 @@
CORE
CORE broken-smt-backend
main.c
--floatbv --no-simplify
^EXIT=0$
Expand Down
2 changes: 1 addition & 1 deletion regression/cbmc/Float-to-int1/test.desc
Original file line number Diff line number Diff line change
@@ -1,4 +1,4 @@
CORE
CORE broken-smt-backend
main.c
--floatbv
^EXIT=0$
Expand Down
2 changes: 1 addition & 1 deletion regression/cbmc/Float-to-int2/test.desc
Original file line number Diff line number Diff line change
@@ -1,4 +1,4 @@
CORE
CORE broken-smt-backend
main.c
--floatbv
^EXIT=0$
Expand Down
2 changes: 1 addition & 1 deletion regression/cbmc/Float-to-int3/test.desc
Original file line number Diff line number Diff line change
@@ -1,4 +1,4 @@
CORE
CORE broken-smt-backend
main.c
--floatbv
^EXIT=0$
Expand Down
2 changes: 1 addition & 1 deletion regression/cbmc/Float-zero-sum1/test.desc
Original file line number Diff line number Diff line change
@@ -1,4 +1,4 @@
CORE
CORE broken-smt-backend
main.c
--floatbv --no-simplify
^EXIT=0$
Expand Down
2 changes: 1 addition & 1 deletion regression/cbmc/Float12/test.desc
Original file line number Diff line number Diff line change
@@ -1,4 +1,4 @@
CORE
CORE broken-smt-backend
main.c

^EXIT=0$
Expand Down
2 changes: 1 addition & 1 deletion regression/cbmc/Float13/test.desc
Original file line number Diff line number Diff line change
@@ -1,4 +1,4 @@
CORE
CORE broken-smt-backend
main.c

^EXIT=0$
Expand Down
2 changes: 1 addition & 1 deletion regression/cbmc/Float20/test.desc
Original file line number Diff line number Diff line change
@@ -1,4 +1,4 @@
CORE
CORE broken-smt-backend
main.c
--floatbv
^EXIT=0$
Expand Down
2 changes: 1 addition & 1 deletion regression/cbmc/Float22/test.desc
Original file line number Diff line number Diff line change
@@ -1,4 +1,4 @@
CORE
CORE broken-smt-backend
main.c
--floatbv
^EXIT=0$
Expand Down
2 changes: 1 addition & 1 deletion regression/cbmc/Float23/test.desc
Original file line number Diff line number Diff line change
@@ -1,4 +1,4 @@
CORE
CORE broken-smt-backend
main.c

^EXIT=0$
Expand Down
2 changes: 1 addition & 1 deletion regression/cbmc/Float3/test.desc
Original file line number Diff line number Diff line change
@@ -1,4 +1,4 @@
CORE
CORE broken-smt-backend
main.c
--floatbv
^EXIT=0$
Expand Down
2 changes: 1 addition & 1 deletion regression/cbmc/Float4/test.desc
Original file line number Diff line number Diff line change
@@ -1,4 +1,4 @@
CORE
CORE broken-smt-backend
main.c
--floatbv
^EXIT=0$
Expand Down
2 changes: 1 addition & 1 deletion regression/cbmc/Float5/test.desc
Original file line number Diff line number Diff line change
@@ -1,4 +1,4 @@
CORE
CORE broken-smt-backend
main.c
--floatbv
^EXIT=0$
Expand Down
2 changes: 1 addition & 1 deletion regression/cbmc/Float6/test.desc
Original file line number Diff line number Diff line change
@@ -1,4 +1,4 @@
CORE
CORE broken-smt-backend
main.c
--floatbv
^EXIT=0$
Expand Down
2 changes: 1 addition & 1 deletion regression/cbmc/Float8/test.desc
Original file line number Diff line number Diff line change
@@ -1,4 +1,4 @@
CORE
CORE broken-smt-backend
main.c

^EXIT=0$
Expand Down
2 changes: 1 addition & 1 deletion regression/cbmc/Linking4/test.desc
Original file line number Diff line number Diff line change
@@ -1,4 +1,4 @@
CORE
CORE broken-smt-backend
link1.c
link2.c
^EXIT=0$
Expand Down
2 changes: 1 addition & 1 deletion regression/cbmc/Linking7/member-name-mismatch.desc
Original file line number Diff line number Diff line change
@@ -1,4 +1,4 @@
CORE
CORE broken-smt-backend
main.c
module2.c
^EXIT=10$
Expand Down
2 changes: 1 addition & 1 deletion regression/cbmc/Linking7/test.desc
Original file line number Diff line number Diff line change
@@ -1,4 +1,4 @@
CORE
CORE broken-smt-backend
main.c
module.c
^EXIT=10$
Expand Down
2 changes: 1 addition & 1 deletion regression/cbmc/Makefile
Original file line number Diff line number Diff line change
Expand Up @@ -4,7 +4,7 @@ test:
@../test.pl -p -c "../../../src/cbmc/cbmc --validate-goto-model" -X smt-backend

test-cprover-smt2:
@../test.pl -p -c "../../../src/cbmc/cbmc --cprover-smt2"
@../test.pl -p -c "../../../src/cbmc/cbmc --cprover-smt2" -X broken-smt-backend
Copy link
Contributor

Choose a reason for hiding this comment

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

The commits are in the wrong order, this should go after the broken-smt-backend tag is introduced

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

I think that's just GitHub displaying them in the wrong order, but let me make this unambiguous by moving this particular change into the same commit that introduces the tag.

Copy link
Contributor

Choose a reason for hiding this comment

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

You can update the commit date to make github display them in the right order git rebase "$(git merge-base HEAD origin/develop)" --ignore-date -x 'git commit --amend -C HEAD --date="$(date -R)" && sleep 1.05' -i


test-paths-lifo:
@../test.pl -p -c "../../../src/cbmc/cbmc --paths lifo" -X thorough-paths -X smt-backend
Expand Down
2 changes: 1 addition & 1 deletion regression/cbmc/Malloc23/test.desc
Original file line number Diff line number Diff line change
@@ -1,4 +1,4 @@
CORE
CORE broken-smt-backend
main.c
--pointer-check
^EXIT=10$
Expand Down
2 changes: 1 addition & 1 deletion regression/cbmc/Multi_Dimensional_Array2/test.desc
Original file line number Diff line number Diff line change
@@ -1,4 +1,4 @@
CORE
CORE broken-smt-backend
main.c

^EXIT=0$
Expand Down
2 changes: 1 addition & 1 deletion regression/cbmc/Pointer_Arithmetic11/test.desc
Original file line number Diff line number Diff line change
@@ -1,4 +1,4 @@
CORE
CORE broken-smt-backend
main.c
--pointer-check --little-endian
^EXIT=0$
Expand Down
2 changes: 1 addition & 1 deletion regression/cbmc/Pointer_byte_extract2/test.desc
Original file line number Diff line number Diff line change
@@ -1,4 +1,4 @@
CORE
CORE broken-smt-backend
main.c
--little-endian
^EXIT=10$
Expand Down
2 changes: 1 addition & 1 deletion regression/cbmc/Pointer_byte_extract5/no-simplify.desc
Original file line number Diff line number Diff line change
@@ -1,4 +1,4 @@
CORE
CORE broken-smt-backend
main.i
--bounds-check --32 --no-simplify
^EXIT=10$
Expand Down
2 changes: 1 addition & 1 deletion regression/cbmc/Pointer_byte_extract5/test.desc
Original file line number Diff line number Diff line change
@@ -1,4 +1,4 @@
CORE
CORE broken-smt-backend
main.i
--bounds-check --32
^EXIT=10$
Expand Down
2 changes: 1 addition & 1 deletion regression/cbmc/Pointer_byte_extract9/test.desc
Original file line number Diff line number Diff line change
@@ -1,4 +1,4 @@
CORE
CORE broken-smt-backend
main.c

^EXIT=10$
Expand Down
2 changes: 1 addition & 1 deletion regression/cbmc/Promotion3/test.desc
Original file line number Diff line number Diff line change
@@ -1,4 +1,4 @@
CORE
CORE broken-smt-backend
main.c

^EXIT=0$
Expand Down
2 changes: 1 addition & 1 deletion regression/cbmc/Quantifiers-assertion/test.desc
Original file line number Diff line number Diff line change
@@ -1,4 +1,4 @@
CORE
CORE broken-smt-backend
main.c

^\*\* Results:$
Expand Down
2 changes: 1 addition & 1 deletion regression/cbmc/Quantifiers-assignment/test.desc
Original file line number Diff line number Diff line change
@@ -1,4 +1,4 @@
CORE
CORE broken-smt-backend
main.c

^\*\* Results:$
Expand Down
2 changes: 1 addition & 1 deletion regression/cbmc/Quantifiers-invalid-var-range/test.desc
Original file line number Diff line number Diff line change
@@ -1,4 +1,4 @@
KNOWNBUG
KNOWNBUG broken-smt-backend
main.c

^\*\* Results:$
Expand Down
2 changes: 1 addition & 1 deletion regression/cbmc/Quantifiers-type/test.desc
Original file line number Diff line number Diff line change
@@ -1,4 +1,4 @@
KNOWNBUG
KNOWNBUG broken-smt-backend
main.c

^\*\* Results:$
Expand Down
2 changes: 1 addition & 1 deletion regression/cbmc/Union_Initialization1/test.desc
Original file line number Diff line number Diff line change
@@ -1,4 +1,4 @@
CORE
CORE broken-smt-backend
main.c

^EXIT=0$
Expand Down
2 changes: 1 addition & 1 deletion regression/cbmc/address_space_size_limit1/test.desc
Original file line number Diff line number Diff line change
@@ -1,4 +1,4 @@
CORE thorough-paths
CORE thorough-paths broken-smt-backend
test.c
--no-simplify --unwind 300 --object-bits 8
too many addressed objects
Expand Down
2 changes: 1 addition & 1 deletion regression/cbmc/array-function-parameters/test.desc
Original file line number Diff line number Diff line change
@@ -1,4 +1,4 @@
CORE
CORE broken-smt-backend
test.c
--function test --min-null-tree-depth 2 --max-nondet-tree-depth 2 --bounds-check
\[test.assertion.1\] line \d+ assertion Test.lists\[0\]->next: SUCCESS
Expand Down
2 changes: 1 addition & 1 deletion regression/cbmc/array-tests/test.desc
Original file line number Diff line number Diff line change
@@ -1,4 +1,4 @@
CORE thorough-paths
CORE thorough-paths broken-smt-backend
main.c

^EXIT=0$
Expand Down
2 changes: 1 addition & 1 deletion regression/cbmc/bounds_check1/test.desc
Original file line number Diff line number Diff line change
@@ -1,4 +1,4 @@
CORE
CORE broken-smt-backend
main.c
--bounds-check --pointer-check
^EXIT=10$
Expand Down
2 changes: 1 addition & 1 deletion regression/cbmc/byte_update2/test.desc
Original file line number Diff line number Diff line change
@@ -1,4 +1,4 @@
CORE
CORE broken-smt-backend
main.c
--little-endian
^EXIT=0$
Expand Down
2 changes: 1 addition & 1 deletion regression/cbmc/byte_update3/test.desc
Original file line number Diff line number Diff line change
@@ -1,4 +1,4 @@
CORE
CORE broken-smt-backend
main.c
--little-endian
^EXIT=0$
Expand Down
2 changes: 1 addition & 1 deletion regression/cbmc/byte_update4/test.desc
Original file line number Diff line number Diff line change
@@ -1,4 +1,4 @@
CORE
CORE broken-smt-backend
main.c
--little-endian
^EXIT=0$
Expand Down
Loading