Move the update functions earlier so that they are common between tools

remove_skip and various other passes compute the updates so moving this
should not effect the behaviour of the program.

Author: martin

src/cbmc/cbmc_parse_options.cpp

@@ -963,6 +963,12 @@ bool cbmc_parse_optionst::process_goto_program(
     string_abstraction(goto_model, log.get_message_handler());
   }
 
+  // recalculate numbers, etc.
+  goto_model.goto_functions.update();
+
+  // add loop ids
+  goto_model.goto_functions.compute_loop_numbers();
+
   // ignore default/user-specified initialization
   // of variables with static lifetime
   if(options.get_bool_option("nondet-static"))
@@ -977,12 +983,6 @@ bool cbmc_parse_optionst::process_goto_program(
   // needs to be done before pointer analysis
   add_failed_symbols(goto_model.symbol_table);
 
-  // recalculate numbers, etc.
-  goto_model.goto_functions.update();
-
-  // add loop ids
-  goto_model.goto_functions.compute_loop_numbers();
-
   if(options.get_bool_option("drop-unused-functions"))
   {
     // Entry point will have been set before and function pointers removed