Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

ecr opcode is no longer supported #513

Merged
merged 2 commits into from
Aug 24, 2023
Merged

ecr opcode is no longer supported #513

merged 2 commits into from
Aug 24, 2023

Conversation

Braqzen
Copy link
Contributor

@Braqzen Braqzen commented Aug 24, 2023

Closes #512

I assume that the 3 instructions are meant to be eck1, ecr1 and ed19 based on the names of the functions that are called but I do not know.

Since the previous ecr opcode was used in all 3 instructions I have put the eck1 opcode in all 3 as well. If that is incorrect then lmk and I'll update.

@Braqzen Braqzen added the documentation Improvements or additions to documentation label Aug 24, 2023
@Braqzen Braqzen self-assigned this Aug 24, 2023
src/fuel-vm/instruction-set.md Outdated Show resolved Hide resolved
src/fuel-vm/instruction-set.md Outdated Show resolved Hide resolved
@Braqzen Braqzen marked this pull request as ready for review August 24, 2023 17:22
@Braqzen Braqzen merged commit 5b1c5dc into master Aug 24, 2023
@Braqzen Braqzen deleted the braqzen-512 branch August 24, 2023 17:23
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
documentation Improvements or additions to documentation
Projects
None yet
Development

Successfully merging this pull request may close these issues.

ecr opcode is no longer supported
2 participants