Eliminate ToArray bounds checks (#81001)

* Remove `RangeIterator.ToArray` bounds check #80633 (comment) * Eliminate additional `ToArray` bounds checks
dotnet · Jan 23, 2023 · a272954 · a272954
1 parent 5bd322b
commit a272954
Show file tree

Hide file tree

Showing 4 changed files with 4 additions and 4 deletions.
diff --git a/src/libraries/System.Linq/src/System/Linq/OrderedEnumerable.SpeedOpt.cs b/src/libraries/System.Linq/src/System/Linq/OrderedEnumerable.SpeedOpt.cs
@@ -21,7 +21,7 @@ public virtual TElement[] ToArray()
 
             TElement[] array = new TElement[count];
             int[] map = SortedMap(buffer);
-            for (int i = 0; i != array.Length; i++)
+            for (int i = 0; i < array.Length; i++)
             {
                 array[i] = buffer._items[map[i]];
             }

diff --git a/src/libraries/System.Linq/src/System/Linq/Partition.SpeedOpt.cs b/src/libraries/System.Linq/src/System/Linq/Partition.SpeedOpt.cs
@@ -254,7 +254,7 @@ public TSource[] ToArray()
                 }
 
                 TSource[] array = new TSource[count];
-                for (int i = 0, curIdx = _minIndexInclusive; i != array.Length; ++i, ++curIdx)
+                for (int i = 0, curIdx = _minIndexInclusive; i < array.Length; ++i, ++curIdx)
                 {
                     array[i] = _source[curIdx];
                 }

diff --git a/src/libraries/System.Linq/src/System/Linq/Range.SpeedOpt.cs b/src/libraries/System.Linq/src/System/Linq/Range.SpeedOpt.cs
@@ -18,7 +18,7 @@ public int[] ToArray()
             {
                 int[] array = new int[_end - _start];
                 int cur = _start;
-                for (int i = 0; i != array.Length; ++i)
+                for (int i = 0; i < array.Length; ++i)
                 {
                     array[i] = cur;
                     ++cur;

diff --git a/src/libraries/System.Linq/src/System/Linq/Select.SpeedOpt.cs b/src/libraries/System.Linq/src/System/Linq/Select.SpeedOpt.cs
@@ -789,7 +789,7 @@ public TResult[] ToArray()
                 }
 
                 TResult[] array = new TResult[count];
-                for (int i = 0, curIdx = _minIndexInclusive; i != array.Length; ++i, ++curIdx)
+                for (int i = 0, curIdx = _minIndexInclusive; i < array.Length; ++i, ++curIdx)
                 {
                     array[i] = _selector(_source[curIdx]);
                 }