Persist address ans slot warmth in CREATE[2] calls. (#1468)

The warm address/slot data was not returned to the calling messageFrame
for CREATE[2] calls as it was for CALL series calls.

Also, improve performance by using more efficient copy constructors and
reusing objects were relevant.

Signed-off-by: Danno Ferrin <danno.ferrin@gmail.com>

ethereum/core/src/main/java/org/hyperledger/besu/ethereum/vm/AbstractCallOperation.java

@@ -213,7 +213,7 @@ public abstract class AbstractCallOperation extends AbstractOperation {
               .returnStack(frame.getReturnStack())
               .build();
       frame.incrementRemainingGas(cost);
-      childFrame.mergeWarmedUpFields(frame);
+      childFrame.copyWarmedUpFields(frame);
 
       frame.getMessageFrameStack().addFirst(childFrame);
       frame.setState(MessageFrame.State.CODE_SUSPENDED);

ethereum/core/src/main/java/org/hyperledger/besu/ethereum/vm/MessageFrame.java

@@ -214,8 +214,8 @@ public class MessageFrame {
   private Gas gasRefund;
   private final Set<Address> selfDestructs;
   private final Map<Address, Wei> refunds;
-  private final Set<Address> warmedUpAddresses;
-  private final Multimap<Address, Bytes32> warmedUpStorage;
+  private Set<Address> warmedUpAddresses;
+  private Multimap<Address, Bytes32> warmedUpStorage;
 
   // Execution Environment fields.
   private final Address recipient;
@@ -858,13 +858,22 @@ public class MessageFrame {
     return !warmedUpStorage.put(address, slot);
   }
 
+  public void copyWarmedUpFields(final MessageFrame parentFrame) {
+    if (parentFrame == this) {
+      return;
+    }
+
+    warmedUpAddresses = new HashSet<>(parentFrame.warmedUpAddresses);
+    warmedUpStorage = HashMultimap.create(parentFrame.warmedUpStorage);
+  }
+
   public void mergeWarmedUpFields(final MessageFrame childFrame) {
     if (childFrame == this) {
       return;
     }
 
-    warmedUpAddresses.addAll(childFrame.warmedUpAddresses);
-    warmedUpStorage.putAll(childFrame.warmedUpStorage);
+    warmedUpAddresses = childFrame.warmedUpAddresses;
+    warmedUpStorage = childFrame.warmedUpStorage;
   }
 
   /**

ethereum/core/src/main/java/org/hyperledger/besu/ethereum/vm/operations/AbstractCreateOperation.java

@@ -149,7 +149,7 @@ public abstract class AbstractCreateOperation extends AbstractOperation {
             .build();
 
     frame.incrementRemainingGas(cost);
-    childFrame.mergeWarmedUpFields(frame);
+    childFrame.copyWarmedUpFields(frame);
 
     frame.getMessageFrameStack().addFirst(childFrame);
     frame.setState(MessageFrame.State.CODE_SUSPENDED);
@@ -165,6 +165,7 @@ public abstract class AbstractCreateOperation extends AbstractOperation {
     frame.popStackItems(getStackItemsConsumed());
 
     if (childFrame.getState() == MessageFrame.State.COMPLETED_SUCCESS) {
+      frame.mergeWarmedUpFields(childFrame);
       frame.pushStackItem(Words.fromAddress(childFrame.getContractAddress()));
     } else {
       frame.setReturnData(childFrame.getOutputData());