Merge pull request #1541 from ltratt/remove_16_byte_case

The 16 byte case here can't possibly be correct in general.
ykjit · Jan 13, 2025 · a25c7a4 · a25c7a4
2 parents 22afe6c + c70f753
commit a25c7a4
Showing 1 changed file with 1 addition and 2 deletions.
diff --git a/ykrt/src/compile/jitc_yk/codegen/x64/deopt.rs b/ykrt/src/compile/jitc_yk/codegen/x64/deopt.rs
@@ -267,8 +267,7 @@ pub(crate) extern "C" fn __yk_deopt(
                                 unsafe { rbp.offset(isize::from(*extra)) }
                             };
                             match size {
-                                // FIXME: Check that 16-byte writes are for float registers only.
-                                16 | 8 => unsafe { ptr::write::<u64>(temp as *mut u64, jitval) },
+                                8 => unsafe { ptr::write::<u64>(temp as *mut u64, jitval) },
                                 4 => unsafe { ptr::write::<u32>(temp as *mut u32, jitval as u32) },
                                 2 => unsafe { ptr::write::<u16>(temp as *mut u16, jitval as u16) },
                                 _ => todo!("{}", size),