diff options
author | Andrew Reynolds <andrew.j.reynolds@gmail.com> | 2019-10-11 14:00:56 -0500 |
---|---|---|
committer | GitHub <noreply@github.com> | 2019-10-11 14:00:56 -0500 |
commit | ef7e56f4217ece19b1caf743e5b1db0d3f549226 (patch) | |
tree | 4cbda5ff1b3ebfeaae68dc0268467503cfb29faa | |
parent | 54449d5a9bd8e0de1a32aa35895f4edae51c5e45 (diff) |
Check that logic is set when synth-fun command is encountered (#3384)
-rw-r--r-- | src/parser/smt2/Smt2.g | 2 | ||||
-rw-r--r-- | src/parser/smt2/smt2.cpp | 1 | ||||
-rw-r--r-- | test/regress/CMakeLists.txt | 1 | ||||
-rw-r--r-- | test/regress/regress0/sygus/no-logic.sy | 14 |
4 files changed, 17 insertions, 1 deletions
diff --git a/src/parser/smt2/Smt2.g b/src/parser/smt2/Smt2.g index dc57be11e..0cce030b0 100644 --- a/src/parser/smt2/Smt2.g +++ b/src/parser/smt2/Smt2.g @@ -585,6 +585,7 @@ sygusCommand returns [std::unique_ptr<CVC4::Command> cmd] ( SYNTH_FUN_V1_TOK { isInv = false; } | SYNTH_INV_V1_TOK { isInv = true; range = EXPR_MANAGER->booleanType(); } ) + { PARSER_STATE->checkThatLogicIsSet(); } symbol[fun,CHECK_UNDECLARED,SYM_VARIABLE] LPAREN_TOK sortedVarList[sortedVarNames] RPAREN_TOK ( sortSymbol[range,CHECK_DECLARED] )? @@ -606,6 +607,7 @@ sygusCommand returns [std::unique_ptr<CVC4::Command> cmd] ( SYNTH_FUN_TOK { isInv = false; } | SYNTH_INV_TOK { isInv = true; range = EXPR_MANAGER->booleanType(); } ) + { PARSER_STATE->checkThatLogicIsSet(); } symbol[fun,CHECK_UNDECLARED,SYM_VARIABLE] LPAREN_TOK sortedVarList[sortedVarNames] RPAREN_TOK ( sortSymbol[range,CHECK_DECLARED] )? diff --git a/src/parser/smt2/smt2.cpp b/src/parser/smt2/smt2.cpp index 47ac2a11b..f9942049a 100644 --- a/src/parser/smt2/smt2.cpp +++ b/src/parser/smt2/smt2.cpp @@ -662,7 +662,6 @@ Smt2::SynthFunFactory::SynthFunFactory( std::vector<std::pair<std::string, CVC4::Type>>& sortedVarNames) : d_smt2(smt2), d_fun(fun), d_isInv(isInv) { - smt2->checkThatLogicIsSet(); if (range.isNull()) { smt2->parseError("Must supply return type for synth-fun."); diff --git a/test/regress/CMakeLists.txt b/test/regress/CMakeLists.txt index 96de4bafc..8d69a1a69 100644 --- a/test/regress/CMakeLists.txt +++ b/test/regress/CMakeLists.txt @@ -909,6 +909,7 @@ set(regress_0_tests regress0/sygus/issue3356-syg-inf-usort.smt2 regress0/sygus/let-ringer.sy regress0/sygus/let-simp.sy + regress0/sygus/no-logic.sy regress0/sygus/no-syntax-test-bool.sy regress0/sygus/no-syntax-test.sy regress0/sygus/parity-AIG-d0.sy diff --git a/test/regress/regress0/sygus/no-logic.sy b/test/regress/regress0/sygus/no-logic.sy new file mode 100644 index 000000000..76584d265 --- /dev/null +++ b/test/regress/regress0/sygus/no-logic.sy @@ -0,0 +1,14 @@ +; COMMAND-LINE: --sygus-out=status --lang=sygus2 +; EXPECT-ERROR: no-logic.sy:7.10: No set-logic command was given before this point. +; EXPECT-ERROR: no-logic.sy:7.10: CVC4 will make all theories available. +; EXPECT-ERROR: no-logic.sy:7.10: Consider setting a stricter logic for (likely) better performance. +; EXPECT-ERROR: no-logic.sy:7.10: To suppress this warning in the future use (set-logic ALL). +; EXPECT: unsat +(synth-fun f ((x Int)) Int + ((Start Int)) + ( + (Start Int (x)) + ) +) + +(check-synth) |