EVM: Prevent Address Creation for EIP-2929 Address Tracking

[holgerd77]

No description provided.

[codecov]

Codecov Report

Merging #3122 (d366228) into master (0443f9b) will increase coverage by 0.00%.
The diff coverage is 66.66%.

Additional details and impacted files

Flag	Coverage Δ
block	88.80% <ø> (ø)
blockchain	92.27% <ø> (ø)
client	87.52% <ø> (ø)
common	98.19% <ø> (ø)
ethash	?
evm	71.93% <66.66%> (+0.05%)	⬆️
statemanager	90.30% <ø> (ø)
trie	89.76% <ø> (ø)
tx	96.36% <ø> (ø)
util	87.57% <ø> (ø)
vm	76.13% <ø> (ø)

Flags with carried forward coverage won't be shown. Click here to find out more.

[jochem-brouwer]

LGTM

Side note: we can maybe further optimize by actually also dropping most addressToBytes. We can internally also just handle the BigInt instead of the Bytes. Then we can likely also change the journal to use BigInt instead of string, and then we can drop bytesToUnprefixedHex in journal.ts too 😄

[holgerd77]

Side note: we can maybe further optimize by actually also dropping most addressToBytes. We can internally also just handle the BigInt instead of the Bytes. Then we can likely also change the journal to use BigInt instead of string, and then we can drop bytesToUnprefixedHex in journal.ts too 😄

I wonder though if we should still do any changes into the BigInt direction, if we think about going full-bytes at some point. 🤔

[jochem-brouwer]

Oh right, good point, we should decide on that first (likely shift to Bytes directly)