Update SetDebug to set both config values (solution 2)

Author: eelanagaraj

core/vm/evm.go

@@ -193,7 +193,10 @@ func (evm *EVM) GetDebug() bool {
 }
 
 func (evm *EVM) SetDebug(value bool) {
+	// Set both of these in sync since they refer to the same config data
+	// and are interchangeably used in the interpreter & evm.
 	evm.Config.Debug = value
+	evm.interpreter.cfg.Debug = value
 }
 
 // Call executes the contract associated with the addr with the given input as

core/vm/interpreter.go

@@ -163,7 +163,7 @@ func (in *EVMInterpreter) Run(contract *Contract, input []byte, readOnly bool) (
 	}()
 	contract.Input = input
 
-	if in.evm.GetDebug() {
+	if in.cfg.Debug {
 		defer func() {
 			if err != nil {
 				if !logged {
@@ -184,7 +184,7 @@ func (in *EVMInterpreter) Run(contract *Contract, input []byte, readOnly bool) (
 		if steps%1000 == 0 && atomic.LoadInt32(&in.evm.abort) != 0 {
 			break
 		}
-		if in.evm.GetDebug() {
+		if in.cfg.Debug {
 			// Capture pre-execution values for tracing.
 			logged, pcCopy, gasCopy = false, pc, contract.Gas
 		}
@@ -253,7 +253,7 @@ func (in *EVMInterpreter) Run(contract *Contract, input []byte, readOnly bool) (
 			mem.Resize(memorySize)
 		}
 
-		if in.evm.GetDebug() {
+		if in.cfg.Debug {
 			in.cfg.Tracer.CaptureState(in.evm, pc, op, gasCopy, cost, callContext, in.returnData, in.evm.depth, err)
 			logged = true
 		}