Skip to content

Commit

Permalink
Merge branch 'master' into tf/debug-context-non-optional-acvm
Browse files Browse the repository at this point in the history
* master:
  chore(debugger): avoid taking ownership of Circuit when building debug context  (#3147)
  • Loading branch information
TomAFrench committed Oct 13, 2023
2 parents 96ea988 + 1b08eb6 commit 4663306
Showing 2 changed files with 3 additions and 3 deletions.
4 changes: 2 additions & 2 deletions tooling/debugger/src/lib.rs
Original file line number Diff line number Diff line change
@@ -21,7 +21,7 @@ struct DebugContext<'backend, B: BlackBoxFunctionSolver> {
acvm: ACVM<'backend, B>,
debug_artifact: DebugArtifact,
foreign_call_executor: ForeignCallExecutor,
circuit: Circuit,
circuit: &'backend Circuit,
show_output: bool,
}

@@ -114,7 +114,7 @@ fn map_command_status(result: SolveResult) -> CommandStatus {

pub fn debug_circuit<B: BlackBoxFunctionSolver>(
blackbox_solver: &B,
circuit: Circuit,
circuit: &Circuit,
debug_artifact: DebugArtifact,
initial_witness: WitnessMap,
show_output: bool,
2 changes: 1 addition & 1 deletion tooling/nargo_cli/src/cli/debug_cmd.rs
Original file line number Diff line number Diff line change
@@ -120,7 +120,7 @@ pub(crate) fn debug_program(

noir_debugger::debug_circuit(
&blackbox_solver,
compiled_program.circuit.clone(),
&compiled_program.circuit,
debug_artifact,
initial_witness,
true,

0 comments on commit 4663306

Please sign in to comment.