unsafe: optimise the memory allocation in Sender.Hash() and Transacti…

…onToMessage()
andyzhang2023 · Aug 7, 2024 · b969512 · b969512
1 parent 1324f36
commit b969512
Show file tree

Hide file tree

Showing 2 changed files with 7 additions and 7 deletions.
diff --git a/core/state_transition.go b/core/state_transition.go
@@ -171,11 +171,11 @@ func TransactionToMessage(tx *types.Transaction, s types.Signer, baseFee *big.In
 	msg := &Message{
 		Nonce:          tx.Nonce(),
 		GasLimit:       tx.Gas(),
-		GasPrice:       new(big.Int).Set(tx.GasPrice()),
-		GasFeeCap:      new(big.Int).Set(tx.GasFeeCap()),
-		GasTipCap:      new(big.Int).Set(tx.GasTipCap()),
+		GasPrice:       tx.GasPriceRef(),
+		GasFeeCap:      tx.GasFeeCapRef(),
+		GasTipCap:      tx.GasTipCapRef(),
 		To:             tx.To(),
-		Value:          tx.Value(),
+		Value:          tx.ValueRef(),
 		Data:           tx.Data(),
 		AccessList:     tx.AccessList(),
 		IsSystemTx:     tx.IsSystemTx(),

diff --git a/core/types/transaction_signing.go b/core/types/transaction_signing.go
@@ -314,11 +314,11 @@ func (s londonSigner) Hash(tx *Transaction) common.Hash {
 		[]interface{}{
 			s.chainId,
 			tx.Nonce(),
-			tx.GasTipCap(),
-			tx.GasFeeCap(),
+			tx.GasTipCapRef(),
+			tx.GasFeeCapRef(),
 			tx.Gas(),
 			tx.To(),
-			tx.Value(),
+			tx.ValueRef(),
 			tx.Data(),
 			tx.AccessList(),
 		})