Add Basic Types + Memory walkthrough to Write your own Cairo VM section #8
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
fmoletta
changed the title
…ction (#12) * Add RunContext * Add VM structure
jrchatruc
approved these changes
Aug 1, 2023
gabrielbosio
approved these changes
Aug 1, 2023
jrchatruc
added a commit
that referenced
this pull request
Aug 3, 2023
* Add roadmap to README (#17) * Add roadmap to README * Small typo * Add felt documentation (#16) * Add Basic Types + Memory walkthrough to Write your own Cairo VM section (#8) * Add non c-specific project guidelines * Copy the rest of the file * Copy + adapt introduction * Copy doc in C repo PR * Replace relocatable structure with go structure * Make Relocatable fields public * Update MaybeRelocatable section with go version * Remove leftover comment * Update Memory structure * Add Insert code (minus temp segments comment) * Add Get code (minus temp segments comment) * Add MemorySegmentManager section * Add maybeRelocatable convenience methods * Add missing language keyword * Fix typo * fmt * Replace hashmap for map * Add `RunContext` & `VirtualMachine` structres to Write your own VM section (#12) * Add RunContext * Add VM structure * First draft OpcodeAssertions * First draft OpcodeAssertions * Testing * TestOpcodeAssertionsResUnconstrained * Testing * Mini improvment in README (#34) Co-authored-by: Pablo Deymonnaz <[email protected]> * Review PR #1 - Removes useless methods - Formats variables name - Removes useless pointers * Review PR #1 - Removes useless methods - Formats variables name - Removes useless pointers - Deletes again mock function - Rename function names * Removed error difference. --------- Co-authored-by: Javier Rodríguez Chatruc <[email protected]> Co-authored-by: fmoletta <[email protected]> Co-authored-by: Pablo Deymonnaz <[email protected]> Co-authored-by: Pablo Deymonnaz <[email protected]>
pablodeymo
added a commit
that referenced
this pull request
Aug 4, 2023
* WIP: Lambdaworks wrapper * Wrapper around lambdaworks library * Execution functionality * compute addresses * Compute Operands * Fix tests * Decode operands tests * Fixing test * Test compute operands * Opcode assertions (#32) * Add roadmap to README (#17) * Add roadmap to README * Small typo * Add felt documentation (#16) * Add Basic Types + Memory walkthrough to Write your own Cairo VM section (#8) * Add non c-specific project guidelines * Copy the rest of the file * Copy + adapt introduction * Copy doc in C repo PR * Replace relocatable structure with go structure * Make Relocatable fields public * Update MaybeRelocatable section with go version * Remove leftover comment * Update Memory structure * Add Insert code (minus temp segments comment) * Add Get code (minus temp segments comment) * Add MemorySegmentManager section * Add maybeRelocatable convenience methods * Add missing language keyword * Fix typo * fmt * Replace hashmap for map * Add `RunContext` & `VirtualMachine` structres to Write your own VM section (#12) * Add RunContext * Add VM structure * First draft OpcodeAssertions * First draft OpcodeAssertions * Testing * TestOpcodeAssertionsResUnconstrained * Testing * Mini improvment in README (#34) Co-authored-by: Pablo Deymonnaz <[email protected]> * Review PR #1 - Removes useless methods - Formats variables name - Removes useless pointers * Review PR #1 - Removes useless methods - Formats variables name - Removes useless pointers - Deletes again mock function - Rename function names * Removed error difference. --------- Co-authored-by: Javier Rodríguez Chatruc <[email protected]> Co-authored-by: fmoletta <[email protected]> Co-authored-by: Pablo Deymonnaz <[email protected]> Co-authored-by: Pablo Deymonnaz <[email protected]> * Fix merge conflicts * merge main * Fix comments * fix typos * Delete circular dependency * Delete unused struct * leave comment code for once deductions is done * Fix felt usage * refactor Add felt in maybe relocatable add * Delete duplicated function * Update pkg/vm/run_context.go Co-authored-by: fmoletta <[email protected]> * Update pkg/vm/memory/relocatable.go Co-authored-by: fmoletta <[email protected]> * Update pkg/vm/memory/relocatable.go Co-authored-by: fmoletta <[email protected]> --------- Co-authored-by: toni-calvin <[email protected]> Co-authored-by: Antonio Calvín García <[email protected]> Co-authored-by: Javier Rodríguez Chatruc <[email protected]> Co-authored-by: fmoletta <[email protected]> Co-authored-by: Pablo Deymonnaz <[email protected]> Co-authored-by: Pablo Deymonnaz <[email protected]>
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
This PR is analogous to lambdaclass/cairo-vm.c#37
Contains code from unmerged PRs #9 #6 #5
Removes code related to negative-indexed relocatables (temp memory) as its better to handle this later on