snarkVM icon indicating copy to clipboard operation
snarkVM copied to clipboard

Split out the compiler crate

Open ljedrz opened this issue 3 years ago • 3 comments

Many users will likely only be interested in the snarkvm-compiler crate, so it makes sense to isolate it.

An alternative would be to introduce a compiler feature for snarkvm (in a fashion similar to the other features that are currently there), but it results in a much larger lockfile (in an empty project using just that dependency): 2561 lines, compared to 1548 when importing an isolated snarkvm-compiler crate.

In addition, the dev-dependencies of snarkvm-parameters are also made more granular, making its compilation faster.

ljedrz avatar Sep 23 '22 12:09 ljedrz

The CI failures are unrelated.

Cc https://github.com/AleoHQ/snarkVM/pull/1092 for the clippy one.

ljedrz avatar Sep 23 '22 12:09 ljedrz

@ljedrz let's wait on this PR until #1100 is merged, which introduces time-sensitive changes for snarkVM

howardwu avatar Sep 24 '22 13:09 howardwu

Sounds good; it's simple enough to redo 👌

ljedrz avatar Sep 24 '22 13:09 ljedrz

Once #1100 is merged, let's move to the testnet3.2 branch and pursue the approach in #1165.

howardwu avatar Oct 15 '22 23:10 howardwu

Done in https://github.com/AleoHQ/snarkVM/pull/1201.

ljedrz avatar Oct 24 '22 07:10 ljedrz