search

NethermindEth / cairo-vm-go

A virtual machine for Cairo written in Go
MIT License
82 stars 50 forks source link

Implement ModBuiltin #666

Closed Sh0g0-1758 closed 2 months ago

Sh0g0-1758 commented 2 months ago

Refer : https://github.dev/lambdaclass/cairo-vm/blob/main/vm/src/vm/runners/builtin_runner/modulo.rs Fixes: #632 Fixes: #634 Fixes: #635

Sh0g0-1758 commented 2 months ago

@cicr99

cicr99 commented 2 months ago

There are conflicts after merging #662 that need to be fixed. This PR creates variables for the name of the builtins and replace all places in the code where the raw string is used with such variables. You'll have to do the same for the new builtin. Also there were some refactors done in the code that affect your branch