rustup: update to nightly-2023-12-16.

LegNeato · LegNeato · commit f987e101d846 · 2024-09-17T11:16:35.000-04:00
diff --git a/CHANGELOG.md b/CHANGELOG.md
@@ -30,7 +30,7 @@ and this project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0
 ## [Unreleased]
 
 ### Changed 🛠
-- [PR#1102](https://github.com/EmbarkStudios/rust-gpu/pull/1102) updated toolchain to `nightly-2023-11-26`
+- [PR#1109](https://github.com/EmbarkStudios/rust-gpu/pull/1109) updated toolchain to `nightly-2023-12-16`
 - [PR#1115](https://github.com/EmbarkStudios/rust-gpu/pull/1115) relaxed `glam` version requirements (from `>=0.22, <=0.24` to `>=0.22, <=0.25`)
 - [PR#1127](https://github.com/EmbarkStudios/rust-gpu/pull/1127) updated `spirv-tools` to `0.10.0`, which follows `vulkan-sdk-1.3.275`
 - [PR#1101](https://github.com/EmbarkStudios/rust-gpu/pull/1101) added `ignore` and `no_run` to documentation to make `cargo test` pass
diff --git a/crates/rustc_codegen_spirv/build.rs b/crates/rustc_codegen_spirv/build.rs
@@ -10,9 +10,9 @@ use std::process::{Command, ExitCode};
 /// `cargo publish`. We need to figure out a way to do this properly, but let's hardcode it for now :/
 //const REQUIRED_RUST_TOOLCHAIN: &str = include_str!("../../rust-toolchain.toml");
 const REQUIRED_RUST_TOOLCHAIN: &str = r#"[toolchain]
-channel = "nightly-2023-11-26"
+channel = "nightly-2023-12-16"
 components = ["rust-src", "rustc-dev", "llvm-tools"]
-# commit_hash = f5dc2653fdd8b5d177b2ccbd84057954340a89fc"#;
+# commit_hash = a96d57bdb6d2bb6d233d7d5aaefc2995ab99be01"#;
 
 fn get_rustc_commit_hash() -> Result<String, Box<dyn Error>> {
     let rustc = std::env::var("RUSTC").unwrap_or_else(|_| String::from("rustc"));
diff --git a/crates/rustc_codegen_spirv/src/attr.rs b/crates/rustc_codegen_spirv/src/attr.rs
@@ -148,13 +148,13 @@ impl AggregatedSpirvAttributes {
     pub fn parse<'tcx>(cx: &CodegenCx<'tcx>, attrs: &'tcx [Attribute]) -> Self {
         let mut aggregated_attrs = Self::default();
 
-        // NOTE(eddyb) `delay_span_bug` ensures that if attribute checking fails
+        // NOTE(eddyb) `span_delayed_bug` ensures that if attribute checking fails
         // to see an attribute error, it will cause an ICE instead.
         for parse_attr_result in crate::symbols::parse_attrs_for_checking(&cx.sym, attrs) {
             let (span, parsed_attr) = match parse_attr_result {
                 Ok(span_and_parsed_attr) => span_and_parsed_attr,
                 Err((span, msg)) => {
-                    cx.tcx.sess.delay_span_bug(span, msg);
+                    cx.tcx.sess.span_delayed_bug(span, msg);
                     continue;
                 }
             };
@@ -166,7 +166,7 @@ impl AggregatedSpirvAttributes {
                 }) => {
                     cx.tcx
                         .sess
-                        .delay_span_bug(span, format!("multiple {category} attributes"));
+                        .span_delayed_bug(span, format!("multiple {category} attributes"));
                 }
             }
         }
diff --git a/crates/rustc_codegen_spirv/src/codegen_cx/constant.rs b/crates/rustc_codegen_spirv/src/codegen_cx/constant.rs
@@ -290,7 +290,8 @@ impl<'tcx> ConstMethods<'tcx> for CodegenCx<'tcx> {
                 }
             }
             Scalar::Ptr(ptr, _) => {
-                let (alloc_id, offset) = ptr.into_parts();
+                let (prov, offset) = ptr.into_parts();
+                let alloc_id = prov.alloc_id();
                 let (base_addr, _base_addr_space) = match self.tcx.global_alloc(alloc_id) {
                     GlobalAlloc::Memory(alloc) => {
                         let pointee = match self.lookup_type(ty) {
diff --git a/crates/rustc_codegen_spirv/src/lib.rs b/crates/rustc_codegen_spirv/src/lib.rs
@@ -500,7 +500,7 @@ pub fn __rustc_codegen_backend() -> Box<dyn CodegenBackend> {
     rustc_driver::install_ice_hook(
         "https://github.com/rust-gpu/rust-gpu/issues/new",
         |handler| {
-            handler.note_without_error(concat!(
+            handler.note(concat!(
                 "`rust-gpu` version `",
                 env!("CARGO_PKG_VERSION"),
                 "`"
diff --git a/crates/rustc_codegen_spirv/src/link.rs b/crates/rustc_codegen_spirv/src/link.rs
@@ -350,7 +350,7 @@ fn do_spirv_opt(
                 }
                 Level::Error => sess.struct_err(msg.message).forget_guarantee(),
                 Level::Warning => sess.struct_warn(msg.message),
-                Level::Info | Level::Debug => sess.struct_note_without_error(msg.message),
+                Level::Info | Level::Debug => sess.struct_note(msg.message),
             };
 
             err.note(format!("module `{}`", filename.display()));
diff --git a/crates/rustc_codegen_spirv/src/linker/mod.rs b/crates/rustc_codegen_spirv/src/linker/mod.rs
@@ -534,7 +534,7 @@ pub fn link(
         }
 
         if any_spirt_bugs {
-            let mut note = sess.struct_note_without_error("SPIR-T bugs were reported");
+            let mut note = sess.struct_note("SPIR-T bugs were reported");
             note.help(format!(
                 "pretty-printed SPIR-T was saved to {}.html",
                 dump_spirt_file_path.as_ref().unwrap().display()
diff --git a/crates/rustc_codegen_spirv/src/linker/test.rs b/crates/rustc_codegen_spirv/src/linker/test.rs
@@ -125,6 +125,7 @@ fn link_with_linker_opts(
         let mut early_error_handler = rustc_session::EarlyErrorHandler::new(
             rustc_session::config::ErrorOutputType::default(),
         );
+        early_error_handler.initialize_checked_jobserver();
         let matches = rustc_driver::handle_options(
             &early_error_handler,
             &["".to_string(), "x.rs".to_string()],
@@ -135,7 +136,7 @@ fn link_with_linker_opts(
 
         rustc_span::create_session_globals_then(sopts.edition, || {
             let mut sess = rustc_session::build_session(
-                &early_error_handler,
+                early_error_handler,
                 sopts,
                 CompilerIO {
                     input: Input::Str {
diff --git a/crates/rustc_codegen_spirv/src/symbols.rs b/crates/rustc_codegen_spirv/src/symbols.rs
@@ -569,7 +569,7 @@ fn parse_local_size_attr(arg: &NestedMetaItem) -> Result<[u32; 3], ParseAttrErro
             }
             Ok(local_size)
         }
-        Some(tuple) if tuple.is_empty() => Err((
+        Some([]) => Err((
             arg.span,
             "#[spirv(compute(threads(x, y, z)))] must have the x dimension specified, trailing ones may be elided".to_string(),
         )),
diff --git a/rust-toolchain.toml b/rust-toolchain.toml
@@ -1,7 +1,7 @@
 [toolchain]
-channel = "nightly-2023-11-26"
+channel = "nightly-2023-12-16"
 components = ["rust-src", "rustc-dev", "llvm-tools"]
-# commit_hash = f5dc2653fdd8b5d177b2ccbd84057954340a89fc
+# commit_hash = a96d57bdb6d2bb6d233d7d5aaefc2995ab99be01
 
 # Whenever changing the nightly channel, update the commit hash above, and make
 # sure to change `REQUIRED_TOOLCHAIN` in `crates/rustc_codegen_spirv/build.rs` also.
diff --git a/tests/ui/dis/panic_builtin_bounds_check.stderr b/tests/ui/dis/panic_builtin_bounds_check.stderr
@@ -10,7 +10,7 @@ OpExtension "SPV_KHR_shader_clock"
 OpMemoryModel Logical Simple
 OpEntryPoint Fragment %2 "main"
 OpExecutionMode %2 OriginUpperLeft
-%3 = OpString "/n[Rust panicked at $SYSROOT/lib/rustlib/src/rust/library/core/src/panicking.rs:189:5]/n index out of bounds: the len is %u but the index is %u/n      in main()/n"
+%3 = OpString "/n[Rust panicked at $SYSROOT/lib/rustlib/src/rust/library/core/src/panicking.rs:208:5]/n index out of bounds: the len is %u but the index is %u/n      in main()/n"
 %4 = OpString $SYSROOT/lib/rustlib/src/rust/library/core/src/panicking.rs"
 %5 = OpString "$DIR/panic_builtin_bounds_check.rs"
 %6 = OpTypeVoid
@@ -29,7 +29,7 @@ OpBranchConditional %13 %15 %16
 %15 = OpLabel
 OpBranch %14
 %16 = OpLabel
-OpLine %4 189 4
+OpLine %4 208 4
 %17 = OpExtInst  %6  %1 1 %3 %11 %10
 OpNoLine
 OpBranch %14
diff --git a/tests/ui/dis/ptr_copy.normal.stderr b/tests/ui/dis/ptr_copy.normal.stderr
@@ -1,13 +1,13 @@
 error: cannot memcpy dynamically sized data
-    --> $CORE_SRC/intrinsics.rs:2776:9
+    --> $CORE_SRC/intrinsics.rs:2793:9
      |
-2776 |         copy(src, dst, count)
+2793 |         copy(src, dst, count)
      |         ^^^^^^^^^^^^^^^^^^^^^
      |
 note: used from within `core::intrinsics::copy::<f32>`
-    --> $CORE_SRC/intrinsics.rs:2762:21
+    --> $CORE_SRC/intrinsics.rs:2779:21
      |
-2762 | pub const unsafe fn copy<T>(src: *const T, dst: *mut T, count: usize) {
+2779 | pub const unsafe fn copy<T>(src: *const T, dst: *mut T, count: usize) {
      |                     ^^^^
 note: called by `ptr_copy::copy_via_raw_ptr`
     --> $DIR/ptr_copy.rs:28:18
diff --git a/tests/ui/dis/ptr_read.stderr b/tests/ui/dis/ptr_read.stderr
@@ -2,7 +2,7 @@
 %4 = OpFunctionParameter  %5
 %6 = OpFunctionParameter  %5
 %7 = OpLabel
-OpLine %8 1200 8
+OpLine %8 1215 8
 %9 = OpLoad  %10  %4
 OpLine %11 7 13
 OpStore %6 %9
diff --git a/tests/ui/dis/ptr_read_method.stderr b/tests/ui/dis/ptr_read_method.stderr
@@ -2,7 +2,7 @@
 %4 = OpFunctionParameter  %5
 %6 = OpFunctionParameter  %5
 %7 = OpLabel
-OpLine %8 1200 8
+OpLine %8 1215 8
 %9 = OpLoad  %10  %4
 OpLine %11 7 13
 OpStore %6 %9
diff --git a/tests/ui/dis/ptr_write.stderr b/tests/ui/dis/ptr_write.stderr
@@ -4,7 +4,7 @@
 %7 = OpLabel
 OpLine %8 7 35
 %9 = OpLoad  %10  %4
-OpLine %11 1400 8
+OpLine %11 1415 8
 OpStore %6 %9
 OpNoLine
 OpReturn
diff --git a/tests/ui/dis/ptr_write_method.stderr b/tests/ui/dis/ptr_write_method.stderr
@@ -4,7 +4,7 @@
 %7 = OpLabel
 OpLine %8 7 37
 %9 = OpLoad  %10  %4
-OpLine %11 1400 8
+OpLine %11 1415 8
 OpStore %6 %9
 OpNoLine
 OpReturn
diff --git a/tests/ui/lang/core/ptr/allocate_const_scalar.stderr b/tests/ui/lang/core/ptr/allocate_const_scalar.stderr
@@ -1,3 +1,12 @@
+warning: the feature `ptr_internals` is internal to the compiler or standard library
+ --> $DIR/allocate_const_scalar.rs:6:12
+  |
+6 | #![feature(ptr_internals)]
+  |            ^^^^^^^^^^^^^
+  |
+  = note: using it is strongly discouraged
+  = note: `#[warn(internal_features)]` on by default
+
 error: pointer has non-null integer address
    |
 note: used from within `allocate_const_scalar::main`
@@ -11,5 +20,5 @@ note: called by `main`
 15 | pub fn main(output: &mut Unique<[u8; 4]>) {
    |        ^^^^
 
-error: aborting due to 1 previous error
+error: aborting due to 1 previous error; 1 warning emitted
 
diff --git a/tests/ui/lang/core/ptr/allocate_vec_like.stderr b/tests/ui/lang/core/ptr/allocate_vec_like.stderr
@@ -0,0 +1,11 @@
+warning: the feature `ptr_internals` is internal to the compiler or standard library
+ --> $DIR/allocate_vec_like.rs:4:12
+  |
+4 | #![feature(ptr_internals)]
+  |            ^^^^^^^^^^^^^
+  |
+  = note: using it is strongly discouraged
+  = note: `#[warn(internal_features)]` on by default
+
+warning: 1 warning emitted
+

Original file line number	Diff line number	Diff line change
`@@ -148,13 +148,13 @@ impl AggregatedSpirvAttributes {`
`148`	`148`	`pub fn parse<'tcx>(cx: &CodegenCx<'tcx>, attrs: &'tcx [Attribute]) -> Self {`
`149`	`149`	`let mut aggregated_attrs = Self::default();`
`150`	`150`
`151`		- // NOTE(eddyb) `delay_span_bug` ensures that if attribute checking fails
	`151`	+ // NOTE(eddyb) `span_delayed_bug` ensures that if attribute checking fails
`152`	`152`	`// to see an attribute error, it will cause an ICE instead.`
`153`	`153`	`for parse_attr_result in crate::symbols::parse_attrs_for_checking(&cx.sym, attrs) {`
`154`	`154`	`let (span, parsed_attr) = match parse_attr_result {`
`155`	`155`	`Ok(span_and_parsed_attr) => span_and_parsed_attr,`
`156`	`156`	`Err((span, msg)) => {`
`157`		`- cx.tcx.sess.delay_span_bug(span, msg);`
	`157`	`+ cx.tcx.sess.span_delayed_bug(span, msg);`
`158`	`158`	`continue;`
`159`	`159`	`}`
`160`	`160`	`};`
`@@ -166,7 +166,7 @@ impl AggregatedSpirvAttributes {`
`166`	`166`	`}) => {`
`167`	`167`	`cx.tcx`
`168`	`168`	`.sess`
`169`		`- .delay_span_bug(span, format!("multiple {category} attributes"));`
	`169`	`+ .span_delayed_bug(span, format!("multiple {category} attributes"));`
`170`	`170`	`}`
`171`	`171`	`}`
`172`	`172`	`}`
Original file line number	Diff line number	Diff line change
`@@ -290,7 +290,8 @@ impl<'tcx> ConstMethods<'tcx> for CodegenCx<'tcx> {`
`290`	`290`	`}`
`291`	`291`	`}`
`292`	`292`	`Scalar::Ptr(ptr, _) => {`
`293`		`- let (alloc_id, offset) = ptr.into_parts();`
	`293`	`+ let (prov, offset) = ptr.into_parts();`
	`294`	`+ let alloc_id = prov.alloc_id();`
`294`	`295`	`let (base_addr, _base_addr_space) = match self.tcx.global_alloc(alloc_id) {`
`295`	`296`	`GlobalAlloc::Memory(alloc) => {`
`296`	`297`	`let pointee = match self.lookup_type(ty) {`
Original file line number	Diff line number	Diff line change
`@@ -350,7 +350,7 @@ fn do_spirv_opt(`
`350`	`350`	`}`
`351`	`351`	`Level::Error => sess.struct_err(msg.message).forget_guarantee(),`
`352`	`352`	`Level::Warning => sess.struct_warn(msg.message),`
`353`		`- Level::Info \| Level::Debug => sess.struct_note_without_error(msg.message),`
	`353`	`+ Level::Info \| Level::Debug => sess.struct_note(msg.message),`
`354`	`354`	`};`
`355`	`355`
`356`	`356`	err.note(format!("module `{}`", filename.display()));
Original file line number	Diff line number	Diff line change
`@@ -534,7 +534,7 @@ pub fn link(`
`534`	`534`	`}`
`535`	`535`
`536`	`536`	`if any_spirt_bugs {`
`537`		`- let mut note = sess.struct_note_without_error("SPIR-T bugs were reported");`
	`537`	`+ let mut note = sess.struct_note("SPIR-T bugs were reported");`
`538`	`538`	`note.help(format!(`
`539`	`539`	`"pretty-printed SPIR-T was saved to {}.html",`
`540`	`540`	`dump_spirt_file_path.as_ref().unwrap().display()`
Original file line number	Diff line number	Diff line change
`@@ -569,7 +569,7 @@ fn parse_local_size_attr(arg: &NestedMetaItem) -> Result<[u32; 3], ParseAttrErro`
`569`	`569`	`}`
`570`	`570`	`Ok(local_size)`
`571`	`571`	`}`
`572`		`- Some(tuple) if tuple.is_empty() => Err((`
	`572`	`+ Some([]) => Err((`
`573`	`573`	`arg.span,`
`574`	`574`	`"#[spirv(compute(threads(x, y, z)))] must have the x dimension specified, trailing ones may be elided".to_string(),`
`575`	`575`	`)),`