ZJIT: Clean up RangeType (#14627)

tekknolagi · web-flow · commit 6b088c876a4e · 2025-09-22T21:27:54.000Z
diff --git a/zjit/src/codegen.rs b/zjit/src/codegen.rs
@@ -1290,7 +1290,7 @@ fn gen_new_range(
     gen_prepare_non_leaf_call(jit, asm, state);
 
     // Call rb_range_new(low, high, flag)
-    asm_ccall!(asm, rb_range_new, low, high, (flag as i64).into())
+    asm_ccall!(asm, rb_range_new, low, high, (flag as i32).into())
 }
 
 fn gen_new_range_fixnum(
diff --git a/zjit/src/hir.rs b/zjit/src/hir.rs
@@ -286,6 +286,7 @@ impl Const {
     }
 }
 
+#[derive(Clone, Copy)]
 pub enum RangeType {
     Inclusive = 0, // include the end value
     Exclusive = 1, // exclude the end value
@@ -306,14 +307,6 @@ impl std::fmt::Debug for RangeType {
     }
 }
 
-impl Clone for RangeType {
-    fn clone(&self) -> Self {
-        *self
-    }
-}
-
-impl Copy for RangeType {}
-
 impl From<u32> for RangeType {
     fn from(flag: u32) -> Self {
         match flag {
@@ -324,12 +317,6 @@ impl From<u32> for RangeType {
     }
 }
 
-impl From<RangeType> for u32 {
-    fn from(range_type: RangeType) -> Self {
-        range_type as u32
-    }
-}
-
 /// Special regex backref symbol types
 #[derive(Debug, Clone, Copy, PartialEq)]
 pub enum SpecialBackrefSymbol {

Original file line number	Diff line number	Diff line change
`@@ -1290,7 +1290,7 @@ fn gen_new_range(`
`1290`	`1290`	`gen_prepare_non_leaf_call(jit, asm, state);`
`1291`	`1291`
`1292`	`1292`	`// Call rb_range_new(low, high, flag)`
`1293`		`- asm_ccall!(asm, rb_range_new, low, high, (flag as i64).into())`
	`1293`	`+ asm_ccall!(asm, rb_range_new, low, high, (flag as i32).into())`
`1294`	`1294`	`}`
`1295`	`1295`
`1296`	`1296`	`fn gen_new_range_fixnum(`
Original file line number	Diff line number	Diff line change
`@@ -286,6 +286,7 @@ impl Const {`
`286`	`286`	`}`
`287`	`287`	`}`
`288`	`288`
	`289`	`+#[derive(Clone, Copy)]`
`289`	`290`	`pub enum RangeType {`
`290`	`291`	`Inclusive = 0, // include the end value`
`291`	`292`	`Exclusive = 1, // exclude the end value`
`@@ -306,14 +307,6 @@ impl std::fmt::Debug for RangeType {`
`306`	`307`	`}`
`307`	`308`	`}`
`308`	`309`
`309`		`-impl Clone for RangeType {`
`310`		`- fn clone(&self) -> Self {`
`311`		`- *self`
`312`		`- }`
`313`		`-}`
`314`		`-`
`315`		`-impl Copy for RangeType {}`
`316`		`-`
`317`	`310`	`impl From<u32> for RangeType {`
`318`	`311`	`fn from(flag: u32) -> Self {`
`319`	`312`	`match flag {`
`@@ -324,12 +317,6 @@ impl From<u32> for RangeType {`
`324`	`317`	`}`
`325`	`318`	`}`
`326`	`319`
`327`		`-impl From<RangeType> for u32 {`
`328`		`- fn from(range_type: RangeType) -> Self {`
`329`		`- range_type as u32`
`330`		`- }`
`331`		`-}`
`332`		`-`
`333`	`320`	`/// Special regex backref symbol types`
`334`	`321`	`#[derive(Debug, Clone, Copy, PartialEq)]`
`335`	`322`	`pub enum SpecialBackrefSymbol {`