Skip to content
This repository was archived by the owner on Apr 9, 2024. It is now read-only.

Commit e07a56d

Browse files
authored
chore(acvm)!: Remove CircuitSimplifer (#421)
1 parent 744e9da commit e07a56d

File tree

4 files changed

+28
-528
lines changed

4 files changed

+28
-528
lines changed

acvm/src/compiler/mod.rs

Lines changed: 1 addition & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -15,8 +15,6 @@ mod transformers;
1515
use optimizers::{GeneralOptimizer, RangeOptimizer};
1616
use transformers::{CSatTransformer, FallbackTransformer, R1CSTransformer};
1717

18-
pub use optimizers::{CircuitSimplifier, SimplifyResult};
19-
2018
#[derive(PartialEq, Eq, Debug, Error)]
2119
pub enum CompileError {
2220
#[error("The blackbox function {0} is not supported by the backend and acvm does not have a fallback implementation")]
@@ -28,7 +26,6 @@ pub fn compile(
2826
acir: Circuit,
2927
np_language: Language,
3028
is_opcode_supported: impl Fn(&Opcode) -> bool,
31-
simplifier: &CircuitSimplifier,
3229
) -> Result<(Circuit, Vec<OpcodeLabel>), CompileError> {
3330
// Instantiate the optimizer.
3431
// Currently the optimizer and reducer are one in the same
@@ -40,7 +37,7 @@ pub fn compile(
4037

4138
// Fallback transformer pass
4239
let (acir, opcode_label) =
43-
FallbackTransformer::transform(acir, is_opcode_supported, simplifier, opcode_labels)?;
40+
FallbackTransformer::transform(acir, is_opcode_supported, opcode_labels)?;
4441

4542
// General optimizer pass
4643
let mut opcodes: Vec<Opcode> = Vec::new();

acvm/src/compiler/optimizers/mod.rs

Lines changed: 0 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -1,8 +1,5 @@
11
mod general;
22
mod redundant_range;
3-
mod simplify;
43

54
pub(crate) use general::GeneralOptimizer;
65
pub(crate) use redundant_range::RangeOptimizer;
7-
// Public as these need to be passed to `acvm::compiler::compile()`
8-
pub use simplify::{CircuitSimplifier, SimplifyResult};

0 commit comments

Comments
 (0)