Skip to content

Commit ad80da6

Browse files
committed
Use Hacker's Delight impl in i64::midpoint instead of wide i128 impl
As LLVM seems to be outperformed by the complexity of signed 128-bits number compared to our Hacker's Delight implementation.[^1] It doesn't seems like it's an improvement for the other sizes[^2], so we let them with the wide implementation. [^1]: https://rust.godbolt.org/z/ravE75EYj [^2]: https://rust.godbolt.org/z/fzr171zKh
1 parent 9fa0146 commit ad80da6

File tree

1 file changed

+2
-2
lines changed

1 file changed

+2
-2
lines changed

library/core/src/num/mod.rs

+2-2
Original file line numberDiff line numberDiff line change
@@ -429,7 +429,7 @@ impl i64 {
429429
from_xe_bytes_doc = "",
430430
bound_condition = "",
431431
}
432-
midpoint_impl! { i64, i128, signed }
432+
midpoint_impl! { i64, signed }
433433
}
434434

435435
impl i128 {
@@ -530,7 +530,7 @@ impl isize {
530530
from_xe_bytes_doc = usize_isize_from_xe_bytes_doc!(),
531531
bound_condition = " on 64-bit targets",
532532
}
533-
midpoint_impl! { isize, i128, signed }
533+
midpoint_impl! { isize, signed }
534534
}
535535

536536
/// If the 6th bit is set ascii is lower case.

0 commit comments

Comments
 (0)