Auto merge of rust-lang#118769 - pitaj:range-experiment, r=<try>

Range experiment

Meant for a crater run to see how much breakage we can expect outside of the Iterator methods themselves

https://internals.rust-lang.org/t/pre-rfc-fixing-range-by-2027/19936

Author: bors

Cargo.lock

@@ -3006,9 +3006,9 @@ checksum = "dc375e1527247fe1a97d8b7156678dfe7c1af2fc075c9a4db3690ecd2a148068"
 
 [[package]]
 name = "proc-macro2"
-version = "1.0.63"
+version = "1.0.70"
 source = "registry+https://github.com/rust-lang/crates.io-index"
-checksum = "7b368fba921b0dce7e60f5e04ec15e565b3303972b42bcfde1d0713b881959eb"
+checksum = "39278fbbf5fb4f646ce651690877f89d1c5811a3d4acb27700c1cb3cdb78fd3b"
 dependencies = [
  "unicode-ident",
 ]
@@ -3064,9 +3064,8 @@ checksum = "07589615d719a60c8dd8a4622e7946465dfef20d1a428f969e3443e7386d5f45"
 
 [[package]]
 name = "quote"
-version = "1.0.29"
-source = "registry+https://github.com/rust-lang/crates.io-index"
-checksum = "573015e8ab27661678357f27dc26460738fd2b6c86e46f386fde94cb5d913105"
+version = "1.0.33"
+source = "git+https://github.com/pitaj/quote#44feebf0594b255a511ff20890a7acbf4d6aeed1"
 dependencies = [
  "proc-macro2",
 ]
@@ -3349,11 +3348,10 @@ dependencies = [
 [[package]]
 name = "rustc-rayon"
 version = "0.5.0"
-source = "registry+https://github.com/rust-lang/crates.io-index"
-checksum = "eb81aadc8837ca6ecebe0fe1353f15df83b3b3cc2cf7a8afd571bc22aa121710"
+source = "git+https://github.com/pitaj/rustc-rayon#e76e554512ce25abb48f4118576ede5d7a457918"
 dependencies = [
  "either",
- "rustc-rayon-core",
+ "rustc-rayon-core 0.5.0 (git+https://github.com/pitaj/rustc-rayon)",
 ]
 
 [[package]]
@@ -3368,6 +3366,17 @@ dependencies = [
  "num_cpus",
 ]
 
+[[package]]
+name = "rustc-rayon-core"
+version = "0.5.0"
+source = "git+https://github.com/pitaj/rustc-rayon#e76e554512ce25abb48f4118576ede5d7a457918"
+dependencies = [
+ "crossbeam-channel",
+ "crossbeam-deque",
+ "crossbeam-utils",
+ "num_cpus",
+]
+
 [[package]]
 name = "rustc-semver"
 version = "1.1.0"
@@ -3697,7 +3706,7 @@ dependencies = [
  "portable-atomic",
  "rustc-hash",
  "rustc-rayon",
- "rustc-rayon-core",
+ "rustc-rayon-core 0.5.0 (registry+https://github.com/rust-lang/crates.io-index)",
  "rustc_arena",
  "rustc_graphviz",
  "rustc_index",
@@ -4023,7 +4032,7 @@ version = "0.0.0"
 dependencies = [
  "libloading 0.7.4",
  "rustc-rayon",
- "rustc-rayon-core",
+ "rustc-rayon-core 0.5.0 (registry+https://github.com/rust-lang/crates.io-index)",
  "rustc_ast",
  "rustc_ast_lowering",
  "rustc_ast_passes",
@@ -4188,7 +4197,7 @@ dependencies = [
  "measureme",
  "polonius-engine",
  "rustc-rayon",
- "rustc-rayon-core",
+ "rustc-rayon-core 0.5.0 (registry+https://github.com/rust-lang/crates.io-index)",
  "rustc_apfloat",
  "rustc_arena",
  "rustc_ast",
@@ -4388,7 +4397,7 @@ version = "0.0.0"
 dependencies = [
  "field-offset",
  "measureme",
- "rustc-rayon-core",
+ "rustc-rayon-core 0.5.0 (registry+https://github.com/rust-lang/crates.io-index)",
  "rustc_data_structures",
  "rustc_errors",
  "rustc_hir",
@@ -4407,7 +4416,7 @@ name = "rustc_query_system"
 version = "0.0.0"
 dependencies = [
  "parking_lot 0.12.1",
- "rustc-rayon-core",
+ "rustc-rayon-core 0.5.0 (registry+https://github.com/rust-lang/crates.io-index)",
  "rustc_ast",
  "rustc_data_structures",
  "rustc_errors",

Cargo.toml

@@ -111,5 +111,8 @@ rustc-std-workspace-core = { path = 'library/rustc-std-workspace-core' }
 rustc-std-workspace-alloc = { path = 'library/rustc-std-workspace-alloc' }
 rustc-std-workspace-std = { path = 'library/rustc-std-workspace-std' }
 
+rustc-rayon = { git = "https://github.com/pitaj/rustc-rayon" }
+quote = { git = "https://github.com/pitaj/quote" }
+
 [patch."https://github.com/rust-lang/rust-clippy"]
 clippy_lints = { path = "src/tools/clippy/clippy_lints" }

compiler/rustc_borrowck/src/borrow_set.rs

@@ -161,7 +161,7 @@ impl<'tcx> BorrowSet<'tcx> {
     }
 
     pub(crate) fn indices(&self) -> impl Iterator<Item = BorrowIndex> {
-        BorrowIndex::from_usize(0)..BorrowIndex::from_usize(self.len())
+        (BorrowIndex::from_usize(0)..BorrowIndex::from_usize(self.len())).into_iter()
     }
 
     pub(crate) fn iter_enumerated(&self) -> impl Iterator<Item = (BorrowIndex, &BorrowData<'tcx>)> {

compiler/rustc_borrowck/src/region_infer/mod.rs

@@ -2140,7 +2140,7 @@ impl<'tcx> RegionInferenceContext<'tcx> {
         // we still want to screen for an "interesting" point to
         // highlight (e.g., a call site or something).
         let target_scc = self.constraint_sccs.scc(target_region);
-        let mut range = 0..path.len();
+        let mut range = (0..path.len()).into_iter();
 
         // As noted above, when reporting an error, there is typically a chain of constraints
         // leading from some "source" region which must outlive some "target" region.

compiler/rustc_builtin_macros/src/deriving/cmp/partial_ord.rs

@@ -28,7 +28,7 @@ pub fn expand_deriving_partial_ord(
             // No data, placing the tag check first makes codegen simpler
             0 => true,
             1..=2 => false,
-            _ => (0..dataful.len() - 1).any(|i| {
+            _ => (0..dataful.len() - 1).into_iter().any(|i| {
                 if dataful[i]
                     && let Some(idx) = dataful[i + 1..].iter().position(|v| *v)
                 {

compiler/rustc_const_eval/src/interpret/projection.rs

@@ -99,7 +99,7 @@ pub trait Projectable<'tcx, Prov: Provenance>: Sized + std::fmt::Debug {
 /// A type representing iteration over the elements of an array.
 pub struct ArrayIterator<'tcx, 'a, Prov: Provenance, P: Projectable<'tcx, Prov>> {
     base: &'a P,
-    range: Range<u64>,
+    range: <Range<u64> as IntoIterator>::IntoIter,
     stride: Size,
     field_layout: TyAndLayout<'tcx>,
     _phantom: PhantomData<Prov>, // otherwise it says `Prov` is never used...
@@ -280,7 +280,13 @@ where
         debug!("project_array_fields: {base:?} {len}");
         base.offset(len * stride, self.layout_of(self.tcx.types.unit).unwrap(), self)?;
         // Create the iterator.
-        Ok(ArrayIterator { base, range: 0..len, stride, field_layout, _phantom: PhantomData })
+        Ok(ArrayIterator {
+            base,
+            range: (0..len).into_iter(),
+            stride,
+            field_layout,
+            _phantom: PhantomData,
+        })
     }
 
     /// Subslicing

compiler/rustc_error_codes/src/error_codes/E0562.md

@@ -5,7 +5,7 @@ Erroneous code example:
 
 ```compile_fail,E0562
 fn main() {
-    let count_to_ten: impl Iterator<Item=usize> = 0..10;
+    let count_to_ten: impl IntoIterator<Item=usize> = 0..10;
     // error: `impl Trait` not allowed outside of function and inherent method
     //        return types
     for i in count_to_ten {
@@ -17,7 +17,7 @@ fn main() {
 Make sure `impl Trait` only appears in return-type position.
 
 ```
-fn count_to_n(n: usize) -> impl Iterator<Item=usize> {
+fn count_to_n(n: usize) -> impl IntoIterator<Item=usize> {
     0..n
 }
 

compiler/rustc_metadata/src/rmeta/decoder.rs

@@ -251,7 +251,7 @@ impl<T: ParameterizedOverTcx> LazyValue<T> {
 }
 
 struct DecodeIterator<'a, 'tcx, T> {
-    elem_counter: std::ops::Range<usize>,
+    elem_counter: <std::ops::Range<usize> as IntoIterator>::IntoIter,
     dcx: DecodeContext<'a, 'tcx>,
     _phantom: PhantomData<fn() -> T>,
 }
@@ -294,7 +294,7 @@ impl<T: ParameterizedOverTcx> LazyArray<T> {
     {
         let mut dcx = metadata.decoder(self.position.get());
         dcx.lazy_state = LazyState::NodeStart(self.position);
-        DecodeIterator { elem_counter: (0..self.num_elems), dcx, _phantom: PhantomData }
+        DecodeIterator { elem_counter: (0..self.num_elems).into_iter(), dcx, _phantom: PhantomData }
     }
 }
 

compiler/rustc_middle/src/mir/interpret/allocation/init_mask.rs

@@ -534,7 +534,7 @@ impl InitMaskMaterialized {
             end: Size,
             is_init: bool,
         ) -> Option<Size> {
-            (start..end).find(|&i| init_mask.get(i) == is_init)
+            (start..end).into_iter().find(|&i| init_mask.get(i) == is_init)
         }
 
         let result = find_bit_fast(self, start, end, is_init);

compiler/rustc_parse/src/parser/diagnostics.rs

@@ -2905,7 +2905,7 @@ impl<'a> Parser<'a> {
     }
 
     pub fn is_diff_marker(&mut self, long_kind: &TokenKind, short_kind: &TokenKind) -> bool {
-        (0..3).all(|i| self.look_ahead(i, |tok| tok == long_kind))
+        (0..3).into_iter().all(|i| self.look_ahead(i, |tok| tok == long_kind))
             && self.look_ahead(3, |tok| tok == short_kind)
     }