Skip to content

refactor(evm): precompiles#4891

Merged
mattsse merged 3 commits intofoundry-rs:masterfrom
DaniPopes:refactor/precompiles
May 8, 2023
Merged

refactor(evm): precompiles#4891
mattsse merged 3 commits intofoundry-rs:masterfrom
DaniPopes:refactor/precompiles

Conversation

@DaniPopes
Copy link
Member

@DaniPopes DaniPopes commented May 7, 2023

Motivation

  • Makes declaring precompiles easier and more readable by using a macro
  • Fix: precompile 0x02 is sha256, not keccak, as pointed out in TG
  • Param names taken from evm.codes

Solution

@DaniPopes DaniPopes force-pushed the refactor/precompiles branch from f11119a to c921841 Compare May 7, 2023 17:18
@DaniPopes DaniPopes force-pushed the refactor/precompiles branch from c921841 to e3d2314 Compare May 7, 2023 17:23
@mattsse mattsse merged commit 2760f85 into foundry-rs:master May 8, 2023
@DaniPopes DaniPopes deleted the refactor/precompiles branch May 8, 2023 17:12
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants