feat: improve UsedStateEVMInspector by only storing writes when they change state

[bertmiller]

During simulation we use an EVM inspector to get state that is used by a transaction by looking at the storage slots that a transaction reads and writes from. However, a large % of transactions write to some slots such that they don't end up changing the slot at the end of the transaction. As an example, for reentrancy locks a storage slot goes from 0 -> 1 -> 0, such that although the transaction has written to the slot it would not actually change it at the end.

For both analytics purposes and future bundle merging algorithms we should change how this works to only store writes if there is a change at the end of the slot.

Here is the evm inspector:
https://github.com/flashbots/rbuilder/blob/develop/crates/rbuilder/src/building/evm_inspector.rs

[malik672]

@bertmiller assign me

[ZanCorDX]

isn't this handled here?

rbuilder/crates/rbuilder/src/building/evm_inspector.rs

Line 95 in c78f4a9

// if we write the same value that we read as the first read we don't have a write