fix: dont filter out created+selfdestruct in pre

[mattsse]

should close this one paradigmxyz/reth#19703

previously we were yeeting them from pre here:

revm-inspectors/src/tracing/builder/geth.rs

Lines 378 to 379 in cbf97c9

	// only keep accounts that are not created
	change_type.get(addr).map(|ty| !ty.0.is_created()).unwrap_or(true)

[antoniolocascio]

I still have a question about the line:

revm-inspectors/src/tracing/builder/geth.rs

Lines 378 to 379 in cbf97c9

	// only keep accounts that are not created
	change_type.get(addr).map(|ty| !ty.0.is_created()).unwrap_or(true)

A contract can be deployed at an address with non-zero balance. If that happens (and it doesn't selfdestruct), the pre section should contain the address with only balance. IIUC, this line would still filter it out. Basically, the rule should be to filter out from pre contracts that have been deployed on empty addresses. I don't have an example to test on geth at hand, but should be easy enough to find one.

[mattsse]

right, I think this makes sense

need to write a new test case for this scenario and look at the geth impl again