Upgrade to LLVM 16, again #109474

nikic · 2023-03-22T08:32:59Z

Relative to the previous attempt in #107224:

Update to GCC 8.5 on dist-x86_64-linux, to avoid std::optional ABI-incompatibility between libstdc++ 7 and 8.
Cherry-pick llvm/llvm-project@96df79a.
Cherry-pick llvm/llvm-project@6fc670e.

While we don't use GCC for the LLVM build, we do use its libstdc++, and there has been an std::optional ABI break in this version. This makes the libLLVM.so for LLVM 16 ABI-incompatible with newer libstdc++ versions, which we use on all other builders, and which download-ci-llvm users are likely to use.

Make sure that the loop is not fully unrolled (which allows eliminating the allocas) in LLVM 16 either.

rustbot · 2023-03-22T08:33:05Z

⚠️ Warning ⚠️

These commits modify submodules.

nikic · 2023-03-22T08:33:46Z

@bors try

bors · 2023-03-22T08:34:01Z

⌛ Trying commit 58ac25b with merge f88bcdede620c33762a4d143e28e2d42dfb6195c...

bors · 2023-03-22T11:04:34Z

☀️ Try build successful - checks-actions
Build commit: f88bcdede620c33762a4d143e28e2d42dfb6195c (f88bcdede620c33762a4d143e28e2d42dfb6195c)

nikic · 2023-03-22T12:05:06Z

I've verified that this does indeed fix the ABI incompatibility issue as follows:

Attempt to bootstrap rustc at 4a04d08 with download-ci-llvm.
Check that rustc +stage1 test.ts crashes.
Copy the libLLVM-16-rust-1.70.0-nightly.so from the f88bcdede620c33762a4d143e28e2d42dfb6195c try build into build/x86_64-unknown-linux-gnu/stage1/lib/
Check that rustc +stage1 test.rs no longer crashes.

cuviper · 2023-03-22T17:44:42Z

@bors r+ rollup=never

bors · 2023-03-22T17:44:44Z

📌 Commit 58ac25b has been approved by cuviper

It is now in the queue for this repository.

bors · 2023-03-24T09:32:16Z

⌛ Testing commit 58ac25b with merge 48d67d5ae611433f83904696a186f8e1c7b59604...

bors · 2023-03-24T10:16:03Z

💔 Test failed - checks-actions

nikic · 2023-03-24T11:22:29Z

@bors retry #108227

rust-log-analyzer · 2023-03-24T13:14:12Z

The job i686-mingw-2 failed! Check out the build log: (web) (plain)

Click to see the possible cause of the failure (guessed by this bot)

[2523/3103] Linking CXX static library lib\libLLVMMSP430Disassembler.a
[2524/3103] Linking CXX static library lib\libLLVMMSP430AsmParser.a
[2525/3103] Linking CXX static library lib\libLLVMMipsDisassembler.a
[2526/3103] Linking CXX static library lib\libLLVMNVPTXInfo.a
FAILED: lib/libLLVMNVPTXInfo.a 
cmd.exe /C "cd . && "C:\Program Files\CMake\bin\cmake.exe" -E rm -f lib\libLLVMNVPTXInfo.a && C:\a\rust\rust\mingw32\bin\ar.exe qc lib\libLLVMNVPTXInfo.a  lib/Target/NVPTX/TargetInfo/CMakeFiles/LLVMNVPTXInfo.dir/NVPTXTargetInfo.cpp.obj && C:\a\rust\rust\mingw32\bin\ranlib.exe lib\libLLVMNVPTXInfo.a && cd ."
C:\a\rust\rust\mingw32\bin\ranlib.exe: could not create temporary file whilst writing archive: no more archived files
[2527/3103] Linking CXX static library lib\libLLVMPowerPCInfo.a
[2528/3103] Linking CXX static library lib\libLLVMMipsDesc.a
[2529/3103] Building CXX object lib/TextAPI/CMakeFiles/LLVMTextAPI.dir/TextStubCommon.cpp.obj
[2530/3103] Building CXX object lib/TextAPI/CMakeFiles/LLVMTextAPI.dir/InterfaceFile.cpp.obj
[2530/3103] Building CXX object lib/TextAPI/CMakeFiles/LLVMTextAPI.dir/InterfaceFile.cpp.obj
[2531/3103] Building CXX object lib/Passes/CMakeFiles/LLVMPasses.dir/StandardInstrumentations.cpp.obj
[2532/3103] Building CXX object lib/Passes/CMakeFiles/LLVMPasses.dir/PassBuilderPipelines.cpp.obj
[2533/3103] Building CXX object lib/Passes/CMakeFiles/LLVMPasses.dir/PassBuilder.cpp.obj
ninja: build stopped: subcommand failed.
command did not execute successfully, got: exit code: 1


build script failed, must exit now', C:\Users\runneradmin\.cargo\registry\src\index.crates.io-1cd66030c949c28d\cmake-0.1.48\src\lib.rs:975:5
 finished in 2142.188 seconds
Build completed unsuccessfully in 0:36:55
Build completed unsuccessfully in 0:36:55
make: *** [Makefile:80: ci-mingw-subset-2] Error 1

bors · 2023-03-24T14:35:05Z

⌛ Testing commit 58ac25b with merge 386cffa92673018f1b775c5e930887b0c3834b57...

bors · 2023-03-25T19:55:13Z

⌛ Testing commit 58ac25b with merge 0c61c7a...

bors · 2023-03-25T22:55:47Z

☀️ Test successful - checks-actions
Approved by: cuviper
Pushing 0c61c7a to master...

rust-timer · 2023-03-26T02:13:15Z

Finished benchmarking commit (0c61c7a): comparison URL.

Overall result: ❌✅ regressions and improvements - ACTION NEEDED

Next Steps: If you can justify the regressions found in this perf run, please indicate this with @rustbot label: +perf-regression-triaged along with sufficient written justification. If you cannot justify the regressions please open an issue or create a new PR that fixes the regressions, add a comment linking to the newly created issue or PR, and then add the perf-regression-triaged label to this PR.

@rustbot label: +perf-regression
cc @rust-lang/wg-compiler-performance

Instruction count

This is a highly reliable metric that was used to determine the overall result at the top of this comment.

	mean	range	count
Regressions ❌ (primary)	1.1%	[0.3%, 3.6%]	64
Regressions ❌ (secondary)	0.9%	[0.2%, 2.5%]	23
Improvements ✅ (primary)	-1.0%	[-2.9%, -0.5%]	49
Improvements ✅ (secondary)	-1.1%	[-4.1%, -0.3%]	75
All ❌✅ (primary)	0.2%	[-2.9%, 3.6%]	113

Max RSS (memory usage)

Results

This is a less reliable metric that may be of interest but was not used to determine the overall result at the top of this comment.

	mean	range	count
Regressions ❌ (primary)	2.1%	[0.8%, 3.2%]	9
Regressions ❌ (secondary)	1.9%	[0.5%, 4.7%]	7
Improvements ✅ (primary)	-3.5%	[-6.6%, -1.7%]	3
Improvements ✅ (secondary)	-1.3%	[-2.2%, -0.5%]	7
All ❌✅ (primary)	0.7%	[-6.6%, 3.2%]	12

Cycles

Results

This is a less reliable metric that may be of interest but was not used to determine the overall result at the top of this comment.

	mean	range	count
Regressions ❌ (primary)	2.1%	[1.2%, 3.2%]	18
Regressions ❌ (secondary)	2.4%	[2.4%, 2.4%]	1
Improvements ✅ (primary)	-2.1%	[-5.0%, -1.0%]	92
Improvements ✅ (secondary)	-3.2%	[-9.4%, -0.4%]	61
All ❌✅ (primary)	-1.4%	[-5.0%, 3.2%]	110

…igation_tests_llvm_16, r=cuviper Fix LVI test post LLVM 16 update rust-lang#109474 updated LLVM to 16. This causes the LVI mitigation tests for the `x86_64-fortanix-unknown-sgx` platform to fail. This PR fixes those tests again. cc: `@jethrogb`

rylev · 2023-03-28T13:25:21Z

Given that the perf results are somewhat even (though regressions do win out), I think we sort of have to take this as is. I don't imagine we would revert an LLVM upgrade unless the perf results were really bad.

cc @nikic @cuviper

@rustbot label: +perf-regression-triaged

nnethercote · 2023-03-29T02:27:06Z

There is good news among the perf results. For big changes like this one it's worth looking at cycles and wall-time. If we take the mean of every result (even non-significant ones) we get these changes:

instructions: -0.06%
cycles: -1.16%
wall-time: -1.19%

Which is good!

One weird wrinkle: the profile with the biggest mean improvement is check, which shouldn't be barely using LLVM at all. Maybe there are memory layout effects occurring.

scottmcm · 2023-03-29T02:32:52Z

One weird wrinkle: the profile with the biggest mean improvement is check, which shouldn't be barely using LLVM at all.

Maybe we got lucky and the check part is actually faster thanks to some new optimizations?

nnethercote · 2023-03-29T02:37:37Z

Oh right: if the rustc being measured not only uses LLVM 16 but is also built with LLVM 16 (which I guess is the case for a stage 2 rustc) then yes, that is the most likely explanation.

nikic · 2023-03-29T08:10:07Z

Right, the check improvements should be due to optimization improvements. What's pretty surprising is that relatively minor instruction count improvements map to such a large wall time improvement (based on the detailed statistics, apparently mostly in typeck?)

nnethercote · 2023-03-29T21:58:21Z

Cache misses and branch mispredictions are both down a bit, that could explain some of it.

Pkgsrc changes: * Adjust patches and cargo checksums to new versions. * Adjust to not cross-build to 8.0, due to LLVM using c++17, so adjust USE_LANGUAGES. Upstream changes: Version 1.70.0 (2023-06-01) ========================== Language -------- - [Relax ordering rules for `asm!` operands] (rust-lang/rust#105798) - [Properly allow macro expanded `format_args` invocations to uses captures] (rust-lang/rust#106505) - [Lint ambiguous glob re-exports] (rust-lang/rust#107880) - [Perform const and unsafe checking for expressions in `let _ = expr` position.] (rust-lang/rust#102256) Compiler -------- - [Extend -Cdebuginfo with new options and named aliases] (rust-lang/rust#109808) This provides a smaller version of debuginfo for cases that only need line number information (`-Cdebuginfo=line-tables-only`), which may eventually become the default for `-Cdebuginfo=1`. - [Make `unused_allocation` lint against `Box::new` too] (rust-lang/rust#104363) - [Detect uninhabited types early in const eval] (rust-lang/rust#109435) - [Switch to LLD as default linker for {arm,thumb}v4t-none-eabi] (rust-lang/rust#109721) - [Add tier 3 target `loongarch64-unknown-linux-gnu`] (rust-lang/rust#96971) - [Add tier 3 target for `i586-pc-nto-qnx700` (QNX Neutrino RTOS, version 7.0)] (rust-lang/rust#109173), - [Insert alignment checks for pointer dereferences as debug assertions] (rust-lang/rust#98112) This catches undefined behavior at runtime, and may cause existing code to fail. Refer to Rust's [platform support page][platform-support-doc] for more information on Rust's tiered platform support. Libraries --------- - [Document NonZeroXxx layout guarantees] (rust-lang/rust#94786) - [Windows: make `Command` prefer non-verbatim paths] (rust-lang/rust#96391) - [Implement Default for some alloc/core iterators] (rust-lang/rust#99929) - [Fix handling of trailing bare CR in str::lines] (rust-lang/rust#100311) - [allow negative numeric literals in `concat!`] (rust-lang/rust#106844) - [Add documentation about the memory layout of `Cell`] (rust-lang/rust#106921) - [Use `partial_cmp` to implement tuple `lt`/`le`/`ge`/`gt`] (rust-lang/rust#108157) - [Stabilize `atomic_as_ptr`] (rust-lang/rust#108419) - [Stabilize `nonnull_slice_from_raw_parts`] (rust-lang/rust#97506) - [Partial stabilization of `once_cell`] (rust-lang/rust#105587) - [Stabilize `nonzero_min_max`] (rust-lang/rust#106633) - [Flatten/inline format_args!() and (string and int) literal arguments into format_args!()] (rust-lang/rust#106824) - [Stabilize movbe target feature] (rust-lang/rust#107711) - [don't splice from files into pipes in io::copy] (rust-lang/rust#108283) - [Add a builtin unstable `FnPtr` trait that is implemented for all function pointers] (rust-lang/rust#108080) This extends `Debug`, `Pointer`, `Hash`, `PartialEq`, `Eq`, `PartialOrd`, and `Ord` implementations for function pointers with all ABIs. Stabilized APIs --------------- - [`NonZero*::MIN/MAX`] (https://doc.rust-lang.org/stable/std/num/struct.NonZeroI8.html#associatedconstant.MIN) - [`BinaryHeap::retain`] (https://doc.rust-lang.org/stable/std/collections/struct.BinaryHeap.html#method.retain) - [`Default for std::collections::binary_heap::IntoIter`] (https://doc.rust-lang.org/stable/std/collections/binary_heap/struct.IntoIter.html) - [`Default for std::collections::btree_map::{IntoIter, Iter, IterMut}`] (https://doc.rust-lang.org/stable/std/collections/btree_map/struct.IntoIter.html) - [`Default for std::collections::btree_map::{IntoKeys, Keys}`] (https://doc.rust-lang.org/stable/std/collections/btree_map/struct.IntoKeys.html) - [`Default for std::collections::btree_map::{IntoValues, Values}`] (https://doc.rust-lang.org/stable/std/collections/btree_map/struct.IntoKeys.html) - [`Default for std::collections::btree_map::Range`] (https://doc.rust-lang.org/stable/std/collections/btree_map/struct.Range.html) - [`Default for std::collections::btree_set::{IntoIter, Iter}`] (https://doc.rust-lang.org/stable/std/collections/btree_set/struct.IntoIter.html) - [`Default for std::collections::btree_set::Range`] (https://doc.rust-lang.org/stable/std/collections/btree_set/struct.Range.html) - [`Default for std::collections::linked_list::{IntoIter, Iter, IterMut}`] (https://doc.rust-lang.org/stable/alloc/collections/linked_list/struct.IntoIter.html) - [`Default for std::vec::IntoIter`] (https://doc.rust-lang.org/stable/alloc/vec/struct.IntoIter.html#impl-Default-for-IntoIter%3CT,+A%3E) - [`Default for std::iter::Chain`] (https://doc.rust-lang.org/stable/std/iter/struct.Chain.html) - [`Default for std::iter::Cloned`] (https://doc.rust-lang.org/stable/std/iter/struct.Cloned.html) - [`Default for std::iter::Copied`] (https://doc.rust-lang.org/stable/std/iter/struct.Copied.html) - [`Default for std::iter::Enumerate`] (https://doc.rust-lang.org/stable/std/iter/struct.Enumerate.html) - [`Default for std::iter::Flatten`] (https://doc.rust-lang.org/stable/std/iter/struct.Flatten.html) - [`Default for std::iter::Fuse`] (https://doc.rust-lang.org/stable/std/iter/struct.Fuse.html) - [`Default for std::iter::Rev`] (https://doc.rust-lang.org/stable/std/iter/struct.Rev.html) - [`Default for std::slice::Iter`] (https://doc.rust-lang.org/stable/std/slice/struct.Iter.html) - [`Default for std::slice::IterMut`] (https://doc.rust-lang.org/stable/std/slice/struct.IterMut.html) - [`Rc::into_inner`] (https://doc.rust-lang.org/stable/alloc/rc/struct.Rc.html#method.into_inner) - [`Arc::into_inner`] (https://doc.rust-lang.org/stable/alloc/sync/struct.Arc.html#method.into_inner) - [`std::cell::OnceCell`] (https://doc.rust-lang.org/stable/std/cell/struct.OnceCell.html) - [`Option::is_some_and`] (https://doc.rust-lang.org/stable/std/option/enum.Option.html#method.is_some_and) - [`NonNull::slice_from_raw_parts`] (https://doc.rust-lang.org/stable/std/ptr/struct.NonNull.html#method.slice_from_raw_parts) - [`Result::is_ok_and`] (https://doc.rust-lang.org/stable/std/result/enum.Result.html#method.is_ok_and) - [`Result::is_err_and`] (https://doc.rust-lang.org/stable/std/result/enum.Result.html#method.is_err_and) - [`std::sync::atomic::Atomic*::as_ptr`] (https://doc.rust-lang.org/stable/std/sync/atomic/struct.AtomicU8.html#method.as_ptr) - [`std::io::IsTerminal`] (https://doc.rust-lang.org/stable/std/io/trait.IsTerminal.html) - [`std::os::linux::net::SocketAddrExt`] (https://doc.rust-lang.org/stable/std/os/linux/net/trait.SocketAddrExt.html) - [`std::os::unix::net::UnixDatagram::bind_addr`] (https://doc.rust-lang.org/stable/std/os/unix/net/struct.UnixDatagram.html#method.bind_addr) - [`std::os::unix::net::UnixDatagram::connect_addr`] (https://doc.rust-lang.org/stable/std/os/unix/net/struct.UnixDatagram.html#method.connect_addr) - [`std::os::unix::net::UnixDatagram::send_to_addr`] (https://doc.rust-lang.org/stable/std/os/unix/net/struct.UnixDatagram.html#method.send_to_addr) - [`std::os::unix::net::UnixListener::bind_addr`] (https://doc.rust-lang.org/stable/std/os/unix/net/struct.UnixListener.html#method.bind_addr) - [`std::path::Path::as_mut_os_str`] (https://doc.rust-lang.org/stable/std/path/struct.Path.html#method.as_mut_os_str) - [`std::sync::OnceLock`] (https://doc.rust-lang.org/stable/std/sync/struct.OnceLock.html) Cargo ----- - [Add `CARGO_PKG_README`] (rust-lang/cargo#11645) - [Make `sparse` the default protocol for crates.io] (rust-lang/cargo#11791) - [Accurately show status when downgrading dependencies] (rust-lang/cargo#11839) - [Use registry.default for login/logout] (rust-lang/cargo#11949) - [Stabilize `cargo logout`] (rust-lang/cargo#11950) Misc ---- - [Stabilize rustdoc `--test-run-directory`] (rust-lang/rust#103682) Compatibility Notes ------------------- - [Prevent stable `libtest` from supporting `-Zunstable-options`] (rust-lang/rust#109044) - [Perform const and unsafe checking for expressions in `let _ = expr` position.] (rust-lang/rust#102256) - [WebAssembly targets enable `sign-ext` and `mutable-globals` features in codegen] (rust-lang/rust#109807) This may cause incompatibility with older execution environments. - [Insert alignment checks for pointer dereferences as debug assertions] (rust-lang/rust#98112) This catches undefined behavior at runtime, and may cause existing code to fail. Internal Changes ---------------- These changes do not affect any public interfaces of Rust, but they represent significant improvements to the performance or internals of rustc and related tools. - [Upgrade to LLVM 16] (rust-lang/rust#109474) - [Use SipHash-1-3 instead of SipHash-2-4 for StableHasher] (rust-lang/rust#107925)

https://github.com/rust-lang/rust/releases/tag/1.70.0 (Internal Changes) rust-lang/rust#109474

https://github.com/rust-lang/rust/releases/tag/1.70.0 (Internal Changes) rust-lang/rust#109474 (cherry picked from commit 1318082)

Pkgsrc changes: * Adjust patches and cargo checksums to new versions. * Add support for NetBSD/riscv64. Upstream changes: Version 1.70.0 (2023-06-01) ========================== Language -------- - [Relax ordering rules for `asm!` operands] (rust-lang/rust#105798) - [Properly allow macro expanded `format_args` invocations to uses captures] (rust-lang/rust#106505) - [Lint ambiguous glob re-exports] (rust-lang/rust#107880) - [Perform const and unsafe checking for expressions in `let _ = expr` position.] (rust-lang/rust#102256) Compiler -------- - [Extend -Cdebuginfo with new options and named aliases] (rust-lang/rust#109808) This provides a smaller version of debuginfo for cases that only need line number information (`-Cdebuginfo=line-tables-only`), which may eventually become the default for `-Cdebuginfo=1`. - [Make `unused_allocation` lint against `Box::new` too] (rust-lang/rust#104363) - [Detect uninhabited types early in const eval] (rust-lang/rust#109435) - [Switch to LLD as default linker for {arm,thumb}v4t-none-eabi] (rust-lang/rust#109721) - [Add tier 3 target `loongarch64-unknown-linux-gnu`] (rust-lang/rust#96971) - [Add tier 3 target for `i586-pc-nto-qnx700`(QNX Neutrino RTOS, version 7.0)] (rust-lang/rust#109173), - [Insert alignment checks for pointer dereferences as debug assertions] (rust-lang/rust#98112) This catches undefined behavior at runtime, and may cause existing code to fail. Refer to Rust's [platform support page][platform-support-doc] for more information on Rust's tiered platform support. Libraries --------- - [Document NonZeroXxx layout guarantees] (rust-lang/rust#94786) - [Windows: make `Command` prefer non-verbatim paths] (rust-lang/rust#96391) - [Implement Default for some alloc/core iterators] (rust-lang/rust#99929) - [Fix handling of trailing bare CR in str::lines] (rust-lang/rust#100311) - [allow negative numeric literals in `concat!`] (rust-lang/rust#106844) - [Add documentation about the memory layout of `Cell`] (rust-lang/rust#106921) - [Use `partial_cmp` to implement tuple `lt`/`le`/`ge`/`gt`] (rust-lang/rust#108157) - [Stabilize `atomic_as_ptr`] (rust-lang/rust#108419) - [Stabilize `nonnull_slice_from_raw_parts`] (rust-lang/rust#97506) - [Partial stabilization of `once_cell`] (rust-lang/rust#105587) - [Stabilize `nonzero_min_max`] (rust-lang/rust#106633) - [Flatten/inline format_args!() and (string and int) literal arguments into format_args!()] (rust-lang/rust#106824) - [Stabilize movbe target feature] (rust-lang/rust#107711) - [don't splice from files into pipes in io::copy] (rust-lang/rust#108283) - [Add a builtin unstable `FnPtr` trait that is implemented for all function pointers] (rust-lang/rust#108080) This extends `Debug`, `Pointer`, `Hash`, `PartialEq`, `Eq`, `PartialOrd`, and `Ord` implementations for function pointers with all ABIs. Stabilized APIs --------------- - [`NonZero*::MIN/MAX`] (https://doc.rust-lang.org/stable/std/num/struct.NonZeroI8.html#associatedconstant.MIN) - [`BinaryHeap::retain`] (https://doc.rust-lang.org/stable/std/collections/struct.BinaryHeap.html#method.retain) - [`Default for std::collections::binary_heap::IntoIter`] (https://doc.rust-lang.org/stable/std/collections/binary_heap/struct.IntoIter.html) - [`Default for std::collections::btree_map::{IntoIter, Iter, IterMut}`] (https://doc.rust-lang.org/stable/std/collections/btree_map/struct.IntoIter.html) - [`Default for std::collections::btree_map::{IntoKeys, Keys}`] (https://doc.rust-lang.org/stable/std/collections/btree_map/struct.IntoKeys.html) - [`Default for std::collections::btree_map::{IntoValues, Values}`] (https://doc.rust-lang.org/stable/std/collections/btree_map/struct.IntoKeys.html) - [`Default for std::collections::btree_map::Range`] (https://doc.rust-lang.org/stable/std/collections/btree_map/struct.Range.html) - [`Default for std::collections::btree_set::{IntoIter, Iter}`] (https://doc.rust-lang.org/stable/std/collections/btree_set/struct.IntoIter.html) - [`Default for std::collections::btree_set::Range`] (https://doc.rust-lang.org/stable/std/collections/btree_set/struct.Range.html) - [`Default for std::collections::linked_list::{IntoIter, Iter, IterMut}`] (https://doc.rust-lang.org/stable/alloc/collections/linked_list/struct.IntoIter.html) - [`Default for std::vec::IntoIter`] (https://doc.rust-lang.org/stable/alloc/vec/struct.IntoIter.html#impl-Default-for-IntoIter%3CT,+A%3E) - [`Default for std::iter::Chain`] (https://doc.rust-lang.org/stable/std/iter/struct.Chain.html) - [`Default for std::iter::Cloned`] (https://doc.rust-lang.org/stable/std/iter/struct.Cloned.html) - [`Default for std::iter::Copied`] (https://doc.rust-lang.org/stable/std/iter/struct.Copied.html) - [`Default for std::iter::Enumerate`] (https://doc.rust-lang.org/stable/std/iter/struct.Enumerate.html) - [`Default for std::iter::Flatten`] (https://doc.rust-lang.org/stable/std/iter/struct.Flatten.html) - [`Default for std::iter::Fuse`] (https://doc.rust-lang.org/stable/std/iter/struct.Fuse.html) - [`Default for std::iter::Rev`] (https://doc.rust-lang.org/stable/std/iter/struct.Rev.html) - [`Default for std::slice::Iter`] (https://doc.rust-lang.org/stable/std/slice/struct.Iter.html) - [`Default for std::slice::IterMut`] (https://doc.rust-lang.org/stable/std/slice/struct.IterMut.html) - [`Rc::into_inner`] (https://doc.rust-lang.org/stable/alloc/rc/struct.Rc.html#method.into_inner) - [`Arc::into_inner`] (https://doc.rust-lang.org/stable/alloc/sync/struct.Arc.html#method.into_inner) - [`std::cell::OnceCell`] (https://doc.rust-lang.org/stable/std/cell/struct.OnceCell.html) - [`Option::is_some_and`] (https://doc.rust-lang.org/stable/std/option/enum.Option.html#method.is_some_and) - [`NonNull::slice_from_raw_parts`] (https://doc.rust-lang.org/stable/std/ptr/struct.NonNull.html#method.slice_from_raw_parts) - [`Result::is_ok_and`] (https://doc.rust-lang.org/stable/std/result/enum.Result.html#method.is_ok_and) - [`Result::is_err_and`] (https://doc.rust-lang.org/stable/std/result/enum.Result.html#method.is_err_and) - [`std::sync::atomic::Atomic*::as_ptr`] (https://doc.rust-lang.org/stable/std/sync/atomic/struct.AtomicU8.html#method.as_ptr) - [`std::io::IsTerminal`] (https://doc.rust-lang.org/stable/std/io/trait.IsTerminal.html) - [`std::os::linux::net::SocketAddrExt`] (https://doc.rust-lang.org/stable/std/os/linux/net/trait.SocketAddrExt.html) - [`std::os::unix::net::UnixDatagram::bind_addr`] (https://doc.rust-lang.org/stable/std/os/unix/net/struct.UnixDatagram.html#method.bind_addr) - [`std::os::unix::net::UnixDatagram::connect_addr`] (https://doc.rust-lang.org/stable/std/os/unix/net/struct.UnixDatagram.html#method.connect_addr) - [`std::os::unix::net::UnixDatagram::send_to_addr`] (https://doc.rust-lang.org/stable/std/os/unix/net/struct.UnixDatagram.html#method.send_to_addr) - [`std::os::unix::net::UnixListener::bind_addr`] (https://doc.rust-lang.org/stable/std/os/unix/net/struct.UnixListener.html#method.bind_addr) - [`std::path::Path::as_mut_os_str`] (https://doc.rust-lang.org/stable/std/path/struct.Path.html#method.as_mut_os_str) - [`std::sync::OnceLock`] (https://doc.rust-lang.org/stable/std/sync/struct.OnceLock.html) Cargo ----- - [Add `CARGO_PKG_README`] (rust-lang/cargo#11645) - [Make `sparse` the default protocol for crates.io] (rust-lang/cargo#11791) - [Accurately show status when downgrading dependencies] (rust-lang/cargo#11839) - [Use registry.default for login/logout] (rust-lang/cargo#11949) - [Stabilize `cargo logout`] (rust-lang/cargo#11950) Misc ---- - [Stabilize rustdoc `--test-run-directory`] (rust-lang/rust#103682) Compatibility Notes ------------------- - [Prevent stable `libtest` from supporting `-Zunstable-options`] (rust-lang/rust#109044) - [Perform const and unsafe checking for expressions in `let _ = expr` position.] (rust-lang/rust#102256) - [WebAssembly targets enable `sign-ext` and `mutable-globals` features in codegen] (rust-lang/rust#109807) This may cause incompatibility with older execution environments. - [Insert alignment checks for pointer dereferences as debug assertions] (rust-lang/rust#98112) This catches undefined behavior at runtime, and may cause existing code to fail. Internal Changes ---------------- These changes do not affect any public interfaces of Rust, but they represent significant improvements to the performance or internals of rustc and related tools. - [Upgrade to LLVM 16] (rust-lang/rust#109474) - [Use SipHash-1-3 instead of SipHash-2-4 for StableHasher] (rust-lang/rust#107925)

This reverts commit 0c61c7a, reversing changes made to 2d429f3.

nikic and others added 3 commits March 22, 2023 09:27

Upgrade to LLVM 16

237f703

Increase array size in array-map.rs

58ac25b

Make sure that the loop is not fully unrolled (which allows eliminating the allocas) in LLVM 16 either.

rustbot assigned cuviper Mar 22, 2023

rustbot added A-testsuite Area: The testsuite used to check the correctness of rustc S-waiting-on-review Status: Awaiting review from the assignee but also interested parties. T-infra Relevant to the infrastructure team, which will review and decide on the PR/issue. labels Mar 22, 2023

nikic changed the title ~~Update to LLVM 16, again~~ Upgrade to LLVM 16, again Mar 22, 2023

nikic mentioned this pull request Mar 22, 2023

Update dist-x86_64-linux to GCC 8.5 #109454

Closed

nikic added the relnotes Marks issues that should be documented in the release notes of the next release. label Mar 22, 2023

bors added S-waiting-on-bors Status: Waiting on bors to run and complete tests. Bors will change the label on completion. and removed S-waiting-on-review Status: Awaiting review from the assignee but also interested parties. labels Mar 22, 2023

bors added S-waiting-on-review Status: Awaiting review from the assignee but also interested parties. and removed S-waiting-on-bors Status: Waiting on bors to run and complete tests. Bors will change the label on completion. labels Mar 24, 2023

bors added S-waiting-on-bors Status: Waiting on bors to run and complete tests. Bors will change the label on completion. and removed S-waiting-on-review Status: Awaiting review from the assignee but also interested parties. labels Mar 24, 2023

bors added the merged-by-bors This PR was explicitly merged by bors. label Mar 25, 2023

bors merged commit 0c61c7a into rust-lang:master Mar 25, 2023

rustbot added this to the 1.70.0 milestone Mar 25, 2023

rustbot added the perf-regression Performance regression. label Mar 26, 2023

compiler-errors mentioned this pull request Mar 26, 2023

Revert "Upgrade to LLVM 16, again" #109622

Closed

raoulstrackx mentioned this pull request Mar 27, 2023

Fix LVI test post LLVM 16 update #109661

Merged

slanterns mentioned this pull request Mar 27, 2023

Initial support for loongarch64-unknown-linux-gnu #96971

Merged

rustbot added the perf-regression-triaged The performance regression has been triaged. label Mar 28, 2023

nnethercote mentioned this pull request Mar 29, 2023

Add triage for 2023-03-28 rust-lang/rustc-perf#1564

Merged

tmiasko mentioned this pull request Mar 31, 2023

Memory sanitizer : false positive regression in nightly-2023-01-18 #107149

Closed

bryangarza mentioned this pull request May 3, 2023

Invalid bitcast error when building rustc (rustc-std-workspace-core) #110472

Closed

pravic mentioned this pull request May 29, 2023

Github rejects rendering of RELEASES.md #112058

Closed

yutannihilation mentioned this pull request Jun 5, 2023

Fix compile error on Windows with Rust 1.70 extendr/rextendr#285

Merged

fgaz added a commit to NixOS/nixpkgs that referenced this pull request Jun 7, 2023

rustc: use llvm_16

1318082

https://github.com/rust-lang/rust/releases/tag/1.70.0 (Internal Changes) rust-lang/rust#109474

erictapen pushed a commit to erictapen/nixpkgs that referenced this pull request Jun 7, 2023

rustc: use llvm_16

1b16ae2

https://github.com/rust-lang/rust/releases/tag/1.70.0 (Internal Changes) rust-lang/rust#109474 (cherry picked from commit 1318082)

clubby789 mentioned this pull request Jun 11, 2023

Miscompilation from .wrapping_offset(isize::MIN).wrapping_offset(isize::MIN) #112526

Closed

pawanjay176 added a commit to pawanjay176/rust that referenced this pull request Jul 31, 2023

Revert "Auto merge of rust-lang#109474 - nikic:llvm-16-again, r=cuviper"

9967fd6

This reverts commit 0c61c7a, reversing changes made to 2d429f3.

lukas-code mentioned this pull request Jul 31, 2023

Recursion using extern "C" causes miscompilation #114312

Closed

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

Upgrade to LLVM 16, again #109474

Upgrade to LLVM 16, again #109474

nikic commented Mar 22, 2023 •

edited

Loading

rustbot commented Mar 22, 2023

nikic commented Mar 22, 2023

bors commented Mar 22, 2023

bors commented Mar 22, 2023

nikic commented Mar 22, 2023 •

edited

Loading

cuviper commented Mar 22, 2023

bors commented Mar 22, 2023

bors commented Mar 24, 2023

bors commented Mar 24, 2023

nikic commented Mar 24, 2023

rust-log-analyzer commented Mar 24, 2023

bors commented Mar 24, 2023

bors commented Mar 25, 2023

bors commented Mar 25, 2023

rust-timer commented Mar 26, 2023

rylev commented Mar 28, 2023

nnethercote commented Mar 29, 2023

scottmcm commented Mar 29, 2023

nnethercote commented Mar 29, 2023

nikic commented Mar 29, 2023

nnethercote commented Mar 29, 2023

Upgrade to LLVM 16, again #109474

Upgrade to LLVM 16, again #109474

Conversation

nikic commented Mar 22, 2023 • edited Loading

rustbot commented Mar 22, 2023

nikic commented Mar 22, 2023

bors commented Mar 22, 2023

bors commented Mar 22, 2023

nikic commented Mar 22, 2023 • edited Loading

cuviper commented Mar 22, 2023

bors commented Mar 22, 2023

bors commented Mar 24, 2023

bors commented Mar 24, 2023

nikic commented Mar 24, 2023

rust-log-analyzer commented Mar 24, 2023

bors commented Mar 24, 2023

bors commented Mar 25, 2023

bors commented Mar 25, 2023

rust-timer commented Mar 26, 2023

Overall result: ❌✅ regressions and improvements - ACTION NEEDED

Instruction count

Max RSS (memory usage)

Cycles

rylev commented Mar 28, 2023

nnethercote commented Mar 29, 2023

scottmcm commented Mar 29, 2023

nnethercote commented Mar 29, 2023

nikic commented Mar 29, 2023

nnethercote commented Mar 29, 2023

nikic commented Mar 22, 2023 •

edited

Loading

nikic commented Mar 22, 2023 •

edited

Loading