Skip to content

Commit

Permalink
chore(ssa refactor): Separate Brillig only logic from Brillig-SSA gen…
Browse files Browse the repository at this point in the history
…eration logic (#1631)

* separate brillig only logic from logic that combines both brillig and ssa

* move operations related to brillig-ssa conversion to brillig_gen

* intermediate step -- make build work by leaking the abstraction

* refactor return instruction -- all of the implementation details about brillig are no longer in brillig_gen

* fix clippy

* add TODO

* move `convert_integer_mod` into BrilligIr

* encapsulate modulo operation in brillig_gen

* process binary instruction

* refactor const instruction

* make code surrounding jumps satisfy the abstraction

* move not instruction

* add foreign call instruction

* move load and store instructions to brillig_ir

* move truncate instruction

* move `mov`, `stop` and `allocate_array` methods

* fix clippy

* remove push_code method in brillig_gen

* move jump and jmpif instructions to brillig_ir

* clean up terminator instruction

* remove allow(deprecated) lint

* refactor document for link and link_with

* use .into so we can get rid of Value import

* add docs

* Update crates/noirc_evaluator/src/brillig/brillig_ir/artifact.rs

Co-authored-by: Álvaro Rodríguez <sirasistant@gmail.com>

* Update crates/noirc_evaluator/src/brillig/brillig_gen.rs

* make code a bit clearer

* fix clippy

---------

Co-authored-by: Álvaro Rodríguez <sirasistant@gmail.com>
  • Loading branch information
kevaundray and sirasistant authored Jun 12, 2023
1 parent 055b892 commit c5da92c
Show file tree
Hide file tree
Showing 8 changed files with 631 additions and 457 deletions.
123 changes: 0 additions & 123 deletions crates/noirc_evaluator/src/brillig/artifact.rs

This file was deleted.

106 changes: 0 additions & 106 deletions crates/noirc_evaluator/src/brillig/binary.rs

This file was deleted.

Loading

0 comments on commit c5da92c

Please sign in to comment.