introduce goto_programt::instructiont::type() and _nonconst()

This introduces two methods to read and write the type of a GOTO program
instruction, and replaces various direct accesses.

Author: kroening

jbmc/src/java_bytecode/remove_exceptions.cpp

@@ -228,7 +228,7 @@ void remove_exceptionst::instrument_exception_handler(
   const goto_programt::targett &instr_it,
   bool may_catch)
 {
-  PRECONDITION(instr_it->type==CATCH);
+  PRECONDITION(instr_it->type() == CATCH);
 
   if(may_catch)
   {
@@ -404,7 +404,7 @@ bool remove_exceptionst::instrument_throw(
   const remove_exceptionst::stack_catcht &stack_catch,
   const std::vector<symbol_exprt> &locals)
 {
-  PRECONDITION(instr_it->type==THROW);
+  PRECONDITION(instr_it->type() == THROW);
 
   const exprt &exc_expr =
     uncaught_exceptions_domaint::get_exception_symbol(instr_it->get_code());
@@ -421,7 +421,7 @@ bool remove_exceptionst::instrument_throw(
     exc_thrown,
     typecast_exprt(exc_expr, exc_thrown.type()));
   // now turn the `throw' into `assignment'
-  instr_it->type=ASSIGN;
+  instr_it->type_nonconst() = ASSIGN;
   instr_it->code_nonconst() = assignment;
 
   return true;
@@ -437,7 +437,7 @@ remove_exceptionst::instrument_function_call(
   const stack_catcht &stack_catch,
   const std::vector<symbol_exprt> &locals)
 {
-  PRECONDITION(instr_it->type==FUNCTION_CALL);
+  PRECONDITION(instr_it->type() == FUNCTION_CALL);
 
   // save the address of the next instruction
   goto_programt::targett next_it=instr_it;
@@ -512,7 +512,7 @@ void remove_exceptionst::instrument_exceptions(
       locals.push_back(instr_it->decl_symbol());
     }
     // Is it a handler push/pop or catch landing-pad?
-    else if(instr_it->type==CATCH)
+    else if(instr_it->type() == CATCH)
     {
       const irep_idt &statement = instr_it->get_code().get_statement();
       // Is it an exception landing pad (start of a catch block)?
@@ -584,12 +584,12 @@ void remove_exceptionst::instrument_exceptions(
       instr_it->turn_into_skip();
       did_something = true;
     }
-    else if(instr_it->type==THROW)
+    else if(instr_it->type() == THROW)
     {
       did_something = instrument_throw(
         function_identifier, goto_program, instr_it, stack_catch, locals);
     }
-    else if(instr_it->type==FUNCTION_CALL)
+    else if(instr_it->type() == FUNCTION_CALL)
     {
       instrumentation_resultt result =
         instrument_function_call(

jbmc/unit/java_bytecode/goto-programs/remove_virtual_functions_without_fallback.cpp

@@ -88,7 +88,7 @@ static goto_programt::const_targett interpret_classid_comparison(
 
   while(pc != program.instructions.end())
   {
-    if(pc->type == GOTO)
+    if(pc->type() == GOTO)
     {
       exprt guard = pc->guard;
       guard = resolve_classid_test(guard, actual_class_id, ns);
@@ -105,7 +105,7 @@ static goto_programt::const_targett interpret_classid_comparison(
         return pc;
       }
     }
-    else if(pc->type == SKIP)
+    else if(pc->type() == SKIP)
     {
       ++pc;
     }

jbmc/unit/java_bytecode/java_bytecode_instrument/virtual_call_null_checks.cpp

@@ -19,7 +19,7 @@ Author: Diffblue Limited.
 static bool is_expected_virtualmethod_call(
   const goto_programt::instructiont &instruction)
 {
-  if(instruction.type != FUNCTION_CALL)
+  if(instruction.type() != FUNCTION_CALL)
     return false;
   const auto &called_function = instruction.call_function();
   if(!can_cast_expr<class_method_descriptor_exprt>(called_function))

jbmc/unit/java_bytecode/java_virtual_functions/virtual_functions.cpp

@@ -30,7 +30,7 @@ void check_function_call(
 
   for(const auto &target : targets)
   {
-    REQUIRE(target->type == goto_program_instruction_typet::FUNCTION_CALL);
+    REQUIRE(target->type() == goto_program_instruction_typet::FUNCTION_CALL);
     REQUIRE(target->call_function().id() == ID_symbol);
     REQUIRE(
       to_symbol_expr(target->call_function()).get_identifier() ==
@@ -69,7 +69,7 @@ SCENARIO(
           {
             // There should be two guarded GOTOs with non-constant guards. One
             // branching for class C and one for class D or O.
-            if(instruction.type == goto_program_instruction_typet::GOTO)
+            if(instruction.type() == goto_program_instruction_typet::GOTO)
             {
               if(instruction.guard.id() == ID_equal)
               {
@@ -146,7 +146,7 @@ SCENARIO(
         {
           irep_idt current_index;
 
-          while(it->type != goto_program_instruction_typet::END_FUNCTION)
+          while(it->type() != goto_program_instruction_typet::END_FUNCTION)
           {
             const source_locationt &loc = it->source_location();
             REQUIRE(loc != source_locationt::nil());
@@ -157,7 +157,7 @@ SCENARIO(
             {
               current_index = new_index;
               // instruction with a new bytecode index begins with ASSERT
-              REQUIRE(it->type == goto_program_instruction_typet::ASSERT);
+              REQUIRE(it->type() == goto_program_instruction_typet::ASSERT);
               // the assertion corresponds to a line coverage goal
               REQUIRE_FALSE(loc.get_basic_block_covered_lines().empty());
             }

src/analyses/custom_bitvector_analysis.cpp

@@ -283,7 +283,7 @@ void custom_bitvector_domaint::transform(
 
   const goto_programt::instructiont &instruction=*from;
 
-  switch(instruction.type)
+  switch(instruction.type())
   {
   case ASSIGN:
     assign_struct_rec(

src/analyses/escape_analysis.cpp

@@ -184,7 +184,7 @@ void escape_domaint::transform(
 
   const goto_programt::instructiont &instruction=*from;
 
-  switch(instruction.type)
+  switch(instruction.type())
   {
   case ASSIGN:
     {
@@ -460,7 +460,7 @@ void escape_analysist::instrument(
 
       const goto_programt::instructiont &instruction=*i_it;
 
-      if(instruction.type == ASSIGN)
+      if(instruction.type() == ASSIGN)
       {
         const exprt &assign_lhs = instruction.assign_lhs();
 
@@ -469,7 +469,7 @@ void escape_analysist::instrument(
         insert_cleanup(
           gf_entry.second, i_it, assign_lhs, cleanup_functions, false, ns);
       }
-      else if(instruction.type == DEAD)
+      else if(instruction.type() == DEAD)
       {
         const auto &dead_symbol = instruction.dead_symbol();
 

src/analyses/global_may_alias.cpp

@@ -107,7 +107,7 @@ void global_may_alias_domaint::transform(
 
   const goto_programt::instructiont &instruction=*from;
 
-  switch(instruction.type)
+  switch(instruction.type())
   {
   case ASSIGN:
   {

src/analyses/goto_rw.cpp

@@ -756,7 +756,7 @@ void goto_rw(
   goto_programt::const_targett target,
   rw_range_sett &rw_set)
 {
-  switch(target->type)
+  switch(target->type())
   {
   case NO_INSTRUCTION_TYPE:
   case INCOMPLETE_GOTO:

src/analyses/interval_domain.cpp

@@ -68,7 +68,7 @@ void interval_domaint::transform(
   locationt to{trace_to->current_location()};
 
   const goto_programt::instructiont &instruction=*from;
-  switch(instruction.type)
+  switch(instruction.type())
   {
   case DECL:
     havoc_rec(instruction.decl_symbol());

src/analyses/invariant_set_domain.cpp

@@ -25,7 +25,7 @@ void invariant_set_domaint::transform(
   locationt from_l(trace_from->current_location());
   locationt to_l(trace_to->current_location());
 
-  switch(from_l->type)
+  switch(from_l->type())
   {
   case GOTO:
     {