remove_returns: do not lose location numbers

Fully replacing an instruction would also invalidate its location
number. This isn't necessary here, and we also didn't run
compute_location_numbers after this change.

Author: tautschnig

Diff for: src/goto-programs/remove_returns.cpp

@@ -130,8 +130,8 @@ void remove_returnst::replace_returns(
 
         // now turn the `return' into `assignment'
         auto labels = std::move(instruction.labels);
-        instruction = goto_programt::make_assignment(
-          assignment, instruction.source_location());
+        instruction.clear(goto_program_instruction_typet::ASSIGN);
+        instruction.code_nonconst() = std::move(assignment);
         instruction.labels = std::move(labels);
       }
       else