Skip to content

Commit ab554f2

Browse files
Petr BauchPetr Bauch
Petr Bauch
authored and
Petr Bauch
committed
Fix after re-basing
1 parent ccace81 commit ab554f2

File tree

2 files changed

+18
-12
lines changed

2 files changed

+18
-12
lines changed

src/goto-programs/goto_program.cpp

+10-10
Original file line numberDiff line numberDiff line change
@@ -683,24 +683,24 @@ void goto_programt::instructiont::validate(
683683
case NO_INSTRUCTION_TYPE:
684684
break;
685685
case GOTO:
686-
DATA_CHECK_WITH_DIAGNOSTICS(
686+
DATA_CHECK_WITH_DIAGNOSTICS(vm,
687687
has_target(),
688688
"goto instruction expects at least one target",
689689
source_location);
690690
// get_target checks that targets.size()==1
691-
DATA_CHECK_WITH_DIAGNOSTICS(
691+
DATA_CHECK_WITH_DIAGNOSTICS(vm,
692692
get_target()->is_target() && get_target()->target_number != 0,
693693
"goto target has to be a target",
694694
source_location);
695695
break;
696696
case ASSUME:
697-
DATA_CHECK_WITH_DIAGNOSTICS(
697+
DATA_CHECK_WITH_DIAGNOSTICS(vm,
698698
targets.empty(),
699699
"assume instruction should not have a target",
700700
source_location);
701701
break;
702702
case ASSERT:
703-
DATA_CHECK_WITH_DIAGNOSTICS(
703+
DATA_CHECK_WITH_DIAGNOSTICS(vm,
704704
targets.empty(),
705705
"assert instruction should not have a target",
706706
source_location);
@@ -722,7 +722,7 @@ void goto_programt::instructiont::validate(
722722
case ATOMIC_END:
723723
break;
724724
case RETURN:
725-
DATA_CHECK_WITH_DIAGNOSTICS(
725+
DATA_CHECK_WITH_DIAGNOSTICS(vm,
726726
code.get_statement() == ID_return,
727727
"return instruction should contain a return statement",
728728
source_location);
@@ -736,29 +736,29 @@ void goto_programt::instructiont::validate(
736736
vm, targets.empty(), "assign instruction should not have a target");
737737
break;
738738
case DECL:
739-
DATA_CHECK_WITH_DIAGNOSTICS(
739+
DATA_CHECK_WITH_DIAGNOSTICS(vm,
740740
code.get_statement() == ID_decl,
741741
"declaration instructions should contain a declaration statement",
742742
source_location);
743-
DATA_CHECK_WITH_DIAGNOSTICS(
743+
DATA_CHECK_WITH_DIAGNOSTICS(vm,
744744
!ns.lookup(to_code_decl(code).get_identifier(), table_symbol),
745745
"declaring unknown symbol: " +
746746
id2string(to_code_decl(code).get_identifier()),
747747
source_location);
748748
break;
749749
case DEAD:
750-
DATA_CHECK_WITH_DIAGNOSTICS(
750+
DATA_CHECK_WITH_DIAGNOSTICS(vm,
751751
code.get_statement() == ID_dead,
752752
"dead instructions should contain a dead statement",
753753
source_location);
754-
DATA_CHECK_WITH_DIAGNOSTICS(
754+
DATA_CHECK_WITH_DIAGNOSTICS(vm,
755755
!ns.lookup(to_code_dead(code).get_identifier(), table_symbol),
756756
"removing unknown symbol: " +
757757
id2string(to_code_dead(code).get_identifier()) + " from scope",
758758
source_location);
759759
break;
760760
case FUNCTION_CALL:
761-
DATA_CHECK_WITH_DIAGNOSTICS(
761+
DATA_CHECK_WITH_DIAGNOSTICS(vm,
762762
code.get_statement() == ID_function_call,
763763
"function call instruction should contain a call statement",
764764
source_location);

src/util/std_code.h

+8-2
Original file line numberDiff line numberDiff line change
@@ -377,8 +377,9 @@ class code_declt:public codet
377377
const validation_modet vm = validation_modet::INVARIANT)
378378
{
379379
DATA_CHECK(
380-
code.operands().size() == 1, "declaration must have one operand");
380+
vm, code.operands().size() == 1, "declaration must have one operand");
381381
DATA_CHECK(
382+
vm,
382383
code.op0().id() == ID_symbol,
383384
"declaring a non-symbol: " +
384385
id2string(to_symbol_expr(code.op0()).get_identifier()));
@@ -448,9 +449,11 @@ class code_deadt:public codet
448449
const validation_modet vm = validation_modet::INVARIANT)
449450
{
450451
DATA_CHECK(
452+
vm,
451453
code.operands().size() == 1,
452454
"removal (code_deadt) must have one operand");
453455
DATA_CHECK(
456+
vm,
454457
code.op0().id() == ID_symbol,
455458
"removing a non-symbol: " +
456459
id2string(to_symbol_expr(code.op0()).get_identifier()) + "from scope");
@@ -1087,6 +1090,7 @@ class code_function_callt:public codet
10871090
const validation_modet vm = validation_modet::INVARIANT)
10881091
{
10891092
DATA_CHECK(
1093+
vm,
10901094
code.operands().size() == 3,
10911095
"function calls must have three operands:\n1) expression to store the "
10921096
"returned values\n2) the function being called\n3) the vector of "
@@ -1102,10 +1106,12 @@ class code_function_callt:public codet
11021106

11031107
if(code.op0().id() == ID_nil)
11041108
DATA_CHECK(
1109+
vm,
11051110
to_code_type(code.op1().type()).return_type().id() == ID_empty,
11061111
"void function should not return value");
11071112
else
11081113
DATA_CHECK(
1114+
vm,
11091115
base_type_eq(
11101116
code.op0().type(), to_code_type(code.op1().type()).return_type(), ns),
11111117
"function returns expression of wrong type");
@@ -1186,7 +1192,7 @@ class code_returnt:public codet
11861192
const codet &code,
11871193
const validation_modet vm = validation_modet::INVARIANT)
11881194
{
1189-
DATA_CHECK(code.operands().size() == 1, "return must have one operand");
1195+
DATA_CHECK(vm, code.operands().size() == 1, "return must have one operand");
11901196
}
11911197
};
11921198

0 commit comments

Comments
 (0)