Skip to content
This repository has been archived by the owner on Mar 25, 2024. It is now read-only.

feat(vm 1.4.1): Remove padding from keccak256 precompile contract #36

Closed
wants to merge 1 commit into from
Closed

feat(vm 1.4.1): Remove padding from keccak256 precompile contract #36

wants to merge 1 commit into from

Conversation

vladbochok
Copy link
Member

What ❔

  • Remove padding from the keccak256 precompile contract

Why ❔

  • VM v1.4.1 apply padding directly in the precompile call

Checklist

  • PR title corresponds to the body of PR (we generate changelog entries from PRs).
  • Tests for the changes have been added / updated.
  • Documentation comments have been added / updated.
  • Code has been formatted via zk fmt and zk lint.

@vladbochok vladbochok mentioned this pull request Oct 4, 2023
Merged
4 tasks
@StanislavBreadless
Copy link
Collaborator

All commits from this PR were moved here: #41

@benceharomi benceharomi deleted the vb-remove-padding-from-keccak256-precompile-contract branch November 1, 2023 10:59
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Reviewers

@StanislavBreadless StanislavBreadless Awaiting requested review from StanislavBreadless StanislavBreadless will be requested when the pull request is marked ready for review StanislavBreadless is a code owner

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@vladbochok @StanislavBreadless