added variants of goto_programt::insert_before/after that take instruction

This avoids duplicate initialization of the inserted instruction.

Author: Daniel Kroening

src/goto-programs/goto_program.h

@@ -615,6 +615,14 @@ class goto_programt
     return instructions.insert(target, instructiont());
   }
 
+  /// Insertion before the instruction pointed-to by the given instruction
+  /// iterator `target`.
+  /// \return newly inserted location
+  targett insert_before(const_targett target, instructiont &&i)
+  {
+    return instructions.insert(target, std::move(i));
+  }
+
   /// Insertion after the instruction pointed-to by the given instruction
   /// iterator `target`.
   /// \return newly inserted location
@@ -623,6 +631,14 @@ class goto_programt
     return instructions.insert(std::next(target), instructiont());
   }
 
+  /// Insertion after the instruction pointed-to by the given instruction
+  /// iterator `target`.
+  /// \return newly inserted location
+  targett insert_after(const_targett target, instructiont &&i)
+  {
+    return instructions.insert(std::next(target), std::move(i));
+  }
+
   /// Appends the given program `p` to `*this`. `p` is destroyed.
   void destructive_append(goto_programt &p)
   {