Auto merge of #2435 - RalfJung:rustup, r=RalfJung

Rustup

Author: bors

rust-version

@@ -1 +1 @@
-41419e70366962c9a878bfe673ef4df38db6f7f1
+35a061724802377a21fc6dac1ebcbb9b8d1f558a

tests/fail/dangling_pointers/null_pointer_write_zst.rs

@@ -1,11 +1,11 @@
 // Some optimizations remove ZST accesses, thus masking this UB.
 //@compile-flags: -Zmir-opt-level=0
-//@error-pattern: memory access failed: null pointer is a dangling pointer
 
 #[allow(deref_nullptr)]
 fn main() {
     // Not using the () type here, as writes of that type do not even have MIR generated.
     // Also not assigning directly as that's array initialization, not assignment.
     let zst_val = [1u8; 0];
     unsafe { std::ptr::null_mut::<[u8; 0]>().write(zst_val) };
+    //~^ERROR: memory access failed: null pointer is a dangling pointer
 }

tests/fail/dangling_pointers/null_pointer_write_zst.stderr

@@ -1,19 +1,13 @@
 error: Undefined Behavior: memory access failed: null pointer is a dangling pointer (it has no provenance)
-  --> RUSTLIB/core/src/ptr/mod.rs:LL:CC
+  --> $DIR/null_pointer_write_zst.rs:LL:CC
    |
-LL |         copy_nonoverlapping(&src as *const T, dst, 1);
-   |         ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^ memory access failed: null pointer is a dangling pointer (it has no provenance)
+LL |     unsafe { std::ptr::null_mut::<[u8; 0]>().write(zst_val) };
+   |              ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^ memory access failed: null pointer is a dangling pointer (it has no provenance)
    |
    = help: this indicates a bug in the program: it performed an invalid operation, and caused Undefined Behavior
    = help: see https://doc.rust-lang.org/nightly/reference/behavior-considered-undefined.html for further information
    = note: backtrace:
-   = note: inside `std::ptr::write::<[u8; 0]>` at RUSTLIB/core/src/ptr/mod.rs:LL:CC
-   = note: inside `std::ptr::mut_ptr::<impl *mut [u8; 0]>::write` at RUSTLIB/core/src/ptr/mut_ptr.rs:LL:CC
-note: inside `main` at $DIR/null_pointer_write_zst.rs:LL:CC
-  --> $DIR/null_pointer_write_zst.rs:LL:CC
-   |
-LL |     unsafe { std::ptr::null_mut::<[u8; 0]>().write(zst_val) };
-   |              ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
+   = note: inside `main` at $DIR/null_pointer_write_zst.rs:LL:CC
 
 note: some details are omitted, run with `MIRIFLAGS=-Zmiri-backtrace=full` for a verbose backtrace
 

tests/fail/data_race/atomic_read_na_write_race1.rs

@@ -1,10 +1,8 @@
 // We want to control preemption here.
 //@compile-flags: -Zmiri-preemption-rate=0
 //@ignore-target-windows: Concurrency on Windows is not supported yet.
-#![feature(core_intrinsics)]
 
-use std::intrinsics;
-use std::sync::atomic::AtomicUsize;
+use std::sync::atomic::{AtomicUsize, Ordering};
 use std::thread::spawn;
 
 #[derive(Copy, Clone)]
@@ -23,8 +21,7 @@ pub fn main() {
         });
 
         let j2 = spawn(move || {
-            //Equivalent to: (&*c.0).load(Ordering::SeqCst)
-            intrinsics::atomic_load_seqcst(c.0 as *mut usize) //~ ERROR: Data race detected between Atomic Load on thread `<unnamed>` and Write on thread `<unnamed>`
+            (&*c.0).load(Ordering::SeqCst) //~ ERROR: Data race detected between Atomic Load on thread `<unnamed>` and Write on thread `<unnamed>`
         });
 
         j1.join().unwrap();

tests/fail/data_race/atomic_read_na_write_race1.stderr

@@ -1,8 +1,8 @@
 error: Undefined Behavior: Data race detected between Atomic Load on thread `<unnamed>` and Write on thread `<unnamed>` at ALLOC
   --> $DIR/atomic_read_na_write_race1.rs:LL:CC
    |
-LL |             intrinsics::atomic_load_seqcst(c.0 as *mut usize)
-   |             ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^ Data race detected between Atomic Load on thread `<unnamed>` and Write on thread `<unnamed>` at ALLOC
+LL |             (&*c.0).load(Ordering::SeqCst)
+   |             ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^ Data race detected between Atomic Load on thread `<unnamed>` and Write on thread `<unnamed>` at ALLOC
    |
    = help: this indicates a bug in the program: it performed an invalid operation, and caused Undefined Behavior
    = help: see https://doc.rust-lang.org/nightly/reference/behavior-considered-undefined.html for further information

tests/fail/data_race/atomic_write_na_read_race2.rs

@@ -1,10 +1,8 @@
 // We want to control preemption here.
 //@compile-flags: -Zmiri-preemption-rate=0
 //@ignore-target-windows: Concurrency on Windows is not supported yet.
-#![feature(core_intrinsics)]
 
-use std::intrinsics::atomic_store;
-use std::sync::atomic::AtomicUsize;
+use std::sync::atomic::{AtomicUsize, Ordering};
 use std::thread::spawn;
 
 #[derive(Copy, Clone)]
@@ -23,8 +21,7 @@ pub fn main() {
         });
 
         let j2 = spawn(move || {
-            //Equivalent to: (&*c.0).store(32, Ordering::SeqCst)
-            atomic_store(c.0 as *mut usize, 32); //~ ERROR: Data race detected between Atomic Store on thread `<unnamed>` and Read on thread `<unnamed>`
+            (&*c.0).store(32, Ordering::SeqCst); //~ ERROR: Data race detected between Atomic Store on thread `<unnamed>` and Read on thread `<unnamed>`
         });
 
         j1.join().unwrap();

tests/fail/data_race/atomic_write_na_read_race2.stderr

@@ -1,7 +1,7 @@
 error: Undefined Behavior: Data race detected between Atomic Store on thread `<unnamed>` and Read on thread `<unnamed>` at ALLOC
   --> $DIR/atomic_write_na_read_race2.rs:LL:CC
    |
-LL |             atomic_store(c.0 as *mut usize, 32);
+LL |             (&*c.0).store(32, Ordering::SeqCst);
    |             ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^ Data race detected between Atomic Store on thread `<unnamed>` and Read on thread `<unnamed>` at ALLOC
    |
    = help: this indicates a bug in the program: it performed an invalid operation, and caused Undefined Behavior

tests/fail/data_race/atomic_write_na_write_race1.rs

@@ -1,10 +1,8 @@
 // We want to control preemption here.
 //@compile-flags: -Zmiri-preemption-rate=0
 //@ignore-target-windows: Concurrency on Windows is not supported yet.
-#![feature(core_intrinsics)]
 
-use std::intrinsics::atomic_store;
-use std::sync::atomic::AtomicUsize;
+use std::sync::atomic::{AtomicUsize, Ordering};
 use std::thread::spawn;
 
 #[derive(Copy, Clone)]
@@ -23,8 +21,7 @@ pub fn main() {
         });
 
         let j2 = spawn(move || {
-            //Equivalent to: (&*c.0).store(64, Ordering::SeqCst)
-            atomic_store(c.0 as *mut usize, 64); //~ ERROR: Data race detected between Atomic Store on thread `<unnamed>` and Write on thread `<unnamed>`
+            (&*c.0).store(64, Ordering::SeqCst); //~ ERROR: Data race detected between Atomic Store on thread `<unnamed>` and Write on thread `<unnamed>`
         });
 
         j1.join().unwrap();

tests/fail/data_race/atomic_write_na_write_race1.stderr

@@ -1,7 +1,7 @@
 error: Undefined Behavior: Data race detected between Atomic Store on thread `<unnamed>` and Write on thread `<unnamed>` at ALLOC
   --> $DIR/atomic_write_na_write_race1.rs:LL:CC
    |
-LL |             atomic_store(c.0 as *mut usize, 64);
+LL |             (&*c.0).store(64, Ordering::SeqCst);
    |             ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^ Data race detected between Atomic Store on thread `<unnamed>` and Write on thread `<unnamed>` at ALLOC
    |
    = help: this indicates a bug in the program: it performed an invalid operation, and caused Undefined Behavior

tests/fail/intrinsics/write_bytes_overflow.rs

@@ -1,9 +1,9 @@
-//@error-pattern: overflow computing total size of `write_bytes`
 use std::mem;
 
 fn main() {
     let mut y = 0;
     unsafe {
         (&mut y as *mut i32).write_bytes(0u8, 1usize << (mem::size_of::<usize>() * 8 - 1));
+        //~^ ERROR: overflow computing total size of `write_bytes`
     }
 }