Skip to content

Commit 5dc9aaf

Browse files
authored
fix: remove redundant clone in store_local calls
1 parent c6c7cdc commit 5dc9aaf

File tree

1 file changed

+5
-5
lines changed
  • crates/cairo-lang-sierra-generator/src/store_variables

1 file changed

+5
-5
lines changed

crates/cairo-lang-sierra-generator/src/store_variables/mod.rs

Lines changed: 5 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -360,7 +360,7 @@ impl<'db> AddStoreVariableStatements<'db> {
360360
) -> VarState {
361361
// Check if this variable should be a local variable.
362362
if let Some(uninitialized_local_var_id) = self.local_variables.get(var) {
363-
self.store_local(var, &uninitialized_local_var_id.clone(), ty);
363+
self.store_local(var, uninitialized_local_var_id.clone(), ty);
364364
VarState::LocalVar
365365
} else {
366366
self.store_temp(known_stack, var, var_on_stack, ty);
@@ -473,7 +473,7 @@ impl<'db> AddStoreVariableStatements<'db> {
473473
ty: &sierra::ids::ConcreteTypeId,
474474
) -> bool {
475475
if let Some(uninitialized_local_var_id) = self.local_variables.get(var) {
476-
self.store_local(var, &uninitialized_local_var_id.clone(), ty);
476+
self.store_local(var, uninitialized_local_var_id.clone(), ty);
477477
return true;
478478
}
479479
false
@@ -486,7 +486,7 @@ impl<'db> AddStoreVariableStatements<'db> {
486486
match var_state {
487487
VarState::Deferred { info: DeferredVariableInfo { ty, .. } }
488488
| VarState::TempVar { ty } => {
489-
self.store_local(var, &uninitialized_local_var_id.clone(), ty);
489+
self.store_local(var, uninitialized_local_var_id.clone(), ty);
490490
*var_state = VarState::LocalVar;
491491
}
492492
VarState::ZeroSizedVar | VarState::LocalVar | VarState::Removed => {}
@@ -531,12 +531,12 @@ impl<'db> AddStoreVariableStatements<'db> {
531531
fn store_local(
532532
&mut self,
533533
var: &sierra::ids::VarId,
534-
uninitialized_local_var_id: &sierra::ids::VarId,
534+
uninitialized_local_var_id: sierra::ids::VarId,
535535
ty: &sierra::ids::ConcreteTypeId,
536536
) {
537537
self.result.push(simple_statement(
538538
store_local_libfunc_id(self.db, ty.clone()),
539-
&[uninitialized_local_var_id.clone(), var.clone()],
539+
&[uninitialized_local_var_id, var.clone()],
540540
std::slice::from_ref(var),
541541
));
542542
}

0 commit comments

Comments
 (0)