Skip to content
New issue

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

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

Already on GitHub? # to your account

fix[venom]: remove duplicate volatile instructions #4263

Merged

Conversation

harkal
Copy link
Collaborator

@harkal harkal commented Sep 27, 2024

What I did

Cleaned up the VOLATILE_INSTRUCTIONS from duplicates

How I did it

How to verify it

Commit message

Remove duplicate `assert`, `assert_unreachable` from `VOLATILE_INSTRUCTIONS` dictionary

Description for the changelog

Cute Animal Picture

Put a link to a cute animal picture inside the parenthesis-->

@harkal harkal marked this pull request as ready for review September 27, 2024 20:03
@charles-cooper charles-cooper changed the title fix[venom]: remove duplicate assert, assert_unreachable fix[venom]: remove duplicate volatile instructions Oct 4, 2024
@charles-cooper charles-cooper merged commit 4f47497 into vyperlang:master Oct 4, 2024
156 checks passed
@charles-cooper charles-cooper deleted the fix/remove_duplicated_instructions branch October 4, 2024 17:35
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants