search

lambdaclass / era_vm

EraVM implementation
MIT License
19 stars 3 forks source link

Fix/refactor ret opcode #147

Closed MarcosNicolau closed 1 month ago

MarcosNicolau commented 1 month ago

This wasn't breaking any test, but when doing reverts and ok, we were not clearing the registers and the context as specified in the spec. Since ret.panic, ret.revert and ret.ok have very similar behavior, I though it was proper to unify them in ret to make sure we ran the same common behaviour in all of them.

MarcosNicolau commented 1 month ago

Depends on #144

juan518munoz commented 1 month ago

when doing reverts and ok, we were not clearing the registers and the context as specified in the spec

Be sure to add a link to the specific part of the spec next time, like this.