Don't clone types

tekknolagi · tekknolagi · commit 005a81b894a6 · 2025-07-07T14:16:35.000-04:00
diff --git a/zjit/src/profile.rs b/zjit/src/profile.rs
@@ -75,24 +75,17 @@ fn profile_insn(profiler: &mut Profiler, opcode: ruby_vminsn_type) {
 /// Profile the Type of top-`n` stack operands
 fn profile_operands(profiler: &mut Profiler, n: usize) {
     let payload = get_or_create_iseq_payload(profiler.iseq);
-    let mut types = if let Some(types) = payload.opnd_types.get(profiler.insn_idx) {
-        types.clone()
-    } else {
-        vec![Empty; n]
-    };
-
+    if payload.opnd_types.len() <= profiler.insn_idx {
+        payload.opnd_types.resize(profiler.insn_idx+1, vec![]);
+    }
+    let types = &mut payload.opnd_types[profiler.insn_idx];
     if types.len() <= n {
         types.resize(n, Empty);
     }
     for i in 0..n {
         let opnd_type = Type::from_value(profiler.peek_at_stack((n - i - 1) as isize));
         types[i] = types[i].union(opnd_type);
     }
-
-    if payload.opnd_types.len() <= profiler.insn_idx {
-        payload.opnd_types.resize(profiler.insn_idx+1, vec![]);
-    }
-    payload.opnd_types[profiler.insn_idx] = types;
 }
 
 /// This is all the data ZJIT stores on an iseq. This will be dynamically allocated by C code
