Improve debug_traceBlock performance

[mattsse]

Describe the feature

currently, trace_block is a loop over transactions and that invoke trace_transaction:

reth/crates/rpc/rpc/src/debug.rs

Lines 107 to 118 in 08bdec9

	while let Some((index, tx)) = transactions.next() {
	let tx_hash = tx.hash;
	let env = EnvWithHandlerCfg {
	env: Env::boxed(
	cfg.cfg_env.clone(),
	block_env.clone(),
	Call::evm_config(this.eth_api()).tx_env(&tx),
	),
	handler_cfg: cfg.handler_cfg,
	};
	let (result, state_changes) = this.trace_transaction(

note how this clones the opts:

reth/crates/rpc/rpc/src/debug.rs

Line 119 in 08bdec9

opts.clone(),

this results in some overhead:

• opts -> TracingConfig conversion

  reth/crates/rpc/rpc/src/debug.rs

  Lines 708 to 710 in 08bdec9

  	let call_config = tracer_config
  	.into_call_config()
  	.map_err(|_| EthApiError::InvalidTracerConfig)?;

• Inspector is not reused: for block level tracing we can reuse the Inspector by fusing it
  https://docs.rs/revm-inspectors/latest/revm_inspectors/tracing/struct.TracingInspector.html#method.fuse
  and because most geth trace results don't consume the traces, we can keep the allocated memory in TracingInspector

https://docs.rs/revm-inspectors/latest/revm_inspectors/tracing/struct.TracingInspector.html#method.geth_builder

TODO

• this should be improved in such a way that the allows us to reuse the internals

this could perhaps be done with a helper type for block tracing that keeps the TracingInspector or similar

Additional context

No response

[JosepBove]

I can take this

[JosepBove]

Ready for review