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

[word lo/hi] cleanup evm_circuit workaround #1491

Merged
merged 3 commits into from
Jun 29, 2023

Conversation

hero78119
Copy link
Member

Description

Clean up workaround introduced in #1435

Issue Link

#1487

Type of change

  • Bug fix (non-breaking change which fixes an issue)

Contents

  • it touch many files just under evm_circuit. Will not cause conflict with other word-lo-hi pr
  • all just cleanup/renaming without any logic change

@hero78119 hero78119 changed the title cleanup workaround [word lo/hi] cleanup evm_circuit workaround Jun 22, 2023
@ChihChengLiang ChihChengLiang linked an issue Jun 22, 2023 that may be closed by this pull request
10 tasks
Copy link
Collaborator

@ChihChengLiang ChihChengLiang left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM.

@KimiWu123 KimiWu123 self-requested a review June 28, 2023 10:25
Copy link
Contributor

@KimiWu123 KimiWu123 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM!

@hero78119 hero78119 merged commit e1350c8 into word-lo-hi Jun 29, 2023
@hero78119 hero78119 deleted the feat/cleanup_wordaround branch June 29, 2023 02:07
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[word lo/hi] optimisation and improvements.
3 participants