Skip to content

Commit

Permalink
Remove clone
Browse files Browse the repository at this point in the history
  • Loading branch information
fmoletta committed Mar 23, 2023
1 parent dd5fb11 commit 1babad2
Showing 1 changed file with 1 addition and 2 deletions.
3 changes: 1 addition & 2 deletions src/vm/runners/cairo_runner.rs
Original file line number Diff line number Diff line change
Expand Up @@ -141,8 +141,7 @@ impl CairoRunner {
if !is_subsequence(&self.program.builtins, &builtin_ordered_list) {
return Err(RunnerError::DisorderedBuiltins);
};
let mut program_builtins: HashSet<BuiltinName> =
self.program.builtins.iter().cloned().collect();
let mut program_builtins: HashSet<&BuiltinName> = self.program.builtins.iter().collect();
let mut builtin_runners = Vec::<BuiltinRunner>::new();

if self.layout.builtins.output {
Expand Down

1 comment on commit 1babad2

@github-actions
Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

⚠️ Performance Alert ⚠️

Possible performance regression was detected for benchmark.
Benchmark result of this commit is worse than the previous benchmark result exceeding threshold 1.30.

Benchmark suite Current: 1babad2 Previous: 77fe09e Ratio
cairo_run(cairo_programs/benchmarks/fibonacci_1000_multirun.json) 112505051 ns/iter (± 13886333) 78887066 ns/iter (± 1882749) 1.43
cairo_run(cairo_programs/benchmarks/set_integration_benchmark.json) 136129221 ns/iter (± 14374732) 104188194 ns/iter (± 222458) 1.31

This comment was automatically generated by workflow using github-action-benchmark.

CC: @unbalancedparentheses

Please sign in to comment.