about summary refs log tree commit diff
path: root/src/libcore/slice
diff options
context:
space:
mode:
authorarthurprs <arthurprs@gmail.com>2018-01-26 12:49:14 +0100
committerarthurprs <arthurprs@gmail.com>2018-01-26 12:49:14 +0100
commit4f7109a42482e266410b23fedc9643a6d76e5fa5 (patch)
tree8d82492efe8eeb755fd3ddf8509696b7b2a03f10 /src/libcore/slice
parent0b56ab0f7b0c5e01611b7ea6a28c77bc09c26275 (diff)
Use the slice length to hint the optimizer
Using the len of the iterator doesn't give the same result.
That's also why we can't generalize it to all TrustedLen iterators.
Diffstat (limited to 'src/libcore/slice')
-rw-r--r--src/libcore/slice/mod.rs6
1 files changed, 4 insertions, 2 deletions
diff --git a/src/libcore/slice/mod.rs b/src/libcore/slice/mod.rs
index d088d4e6634..ac70ae583c7 100644
--- a/src/libcore/slice/mod.rs
+++ b/src/libcore/slice/mod.rs
@@ -1223,7 +1223,8 @@ macro_rules! iterator {
                 P: FnMut(Self::Item) -> bool,
             {
                 // The addition might panic on overflow
-                let n = self.len();
+                // Use the len of the slice to hint optimizer to remove result index bounds check.
+                let n = make_slice!(self.ptr, self.end).len();
                 self.try_fold(0, move |i, x| {
                     if predicate(x) { Err(i) }
                     else { Ok(i + 1) }
@@ -1241,7 +1242,8 @@ macro_rules! iterator {
             {
                 // No need for an overflow check here, because `ExactSizeIterator`
                 // implies that the number of elements fits into a `usize`.
-                let n = self.len();
+                // Use the len of the slice to hint optimizer to remove result index bounds check.
+                let n = make_slice!(self.ptr, self.end).len();
                 self.try_rfold(n, move |i, x| {
                     let i = i - 1;
                     if predicate(x) { Err(i) }