fix some comments, and run_compiler return type

Auto merge of #1406 - RalfJung:miri-as-rustc, r=RalfJung

cargo-miri: never invoke rustc

Always go through 'MIRI_BE_RUSTC=1 miri' instead. This is based on @oli-obk's great idea to add a way to make Miri behave like rustc, which already helped us in https://github.com/rust-lang/miri/pull/1405. Now it means in `cargo-miri` we run *all* crates through the same binary, and use the env var to determine if we compile or interpret them. This makes sure the compiler is consistent.

The `rustc` binary of the current toolchain is now not used at all, only the `miri` binary is. In particular this means we can kill the sysroot consistency check. :)

we do not need to set RUSTFLAGS for xargo any more as miri-as-rustc already uses these flags

fix dead link in README

cargo-miri: never invoke rustc, always go through 'MIRI_BE_RUSTC=1 miri' instead

Auto merge of #1402 - rust-lang:alloc_map_unlock, r=RalfJung

Update to rustc changes

changes needed once https://github.com/rust-lang/rust/pull/71508 gets merged

rustup

Update to rustc changes

Auto merge of #1405 - RalfJung:stage-0, r=RalfJung

make Miri work in rustc bootstrap stage 0

Fixes https://github.com/rust-lang/rust/issues/52856

Typo

adjust default sysroot when being rustc

Also while at it, refactor how we pass the default Miri flags

make Miri work in rustc bootstrap stage 0

Auto merge of #1404 - RalfJung:cargo-miri-host-detect, r=RalfJung

re-do cargo-miri host/target detection logic to match rustbuild

@oli-obk I think that's better than looking at `--emit` like we did before... what do you think?

avoid env::var which requires valid UTF-8

more comment

Wording

Co-authored-by: Oliver Scherer <github35764891676564198441@oli-obk.de>

cargo-miri: honor RUSTC env var

re-do cargo-miri host/target detection logic to match rustbuild

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

bump Rust, fix for renames

bump Rust, fix for renames

Auto merge of #1401 - RalfJung:cargo-miri-help, r=RalfJung

fix cargo-miri intercepting --help/--version arguments

With this, `cargo miri test -- -- --help` should behave as expected and pass `--help` to the test runner.

Thanks to @brson for the report.

fix cargo-miri intercepting --help/--version arguments

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

rustup

rustup

Auto merge of #1397 - brson:tikv-2, r=RalfJung

Add two more TiKV bugs to trophy case

adjust wording

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

rustup; fix error messages

rustup; fix error messages

Add two more TiKV bugs to trophy case

Auto merge of #1396 - RalfJung:float-cast-tests, r=RalfJung

copy some float cast tests from rustc

The more the merrirer!

copy some float cast tests from rustc

Auto merge of #1394 - RalfJung:fs-mode, r=RalfJung

fs shim: check that mode is default

fs shim: check that mode is default

Auto merge of #1393 - RalfJung:arg-count-cleanup, r=RalfJung

Make sure we check argument count everywhere and argument size for non-stub shims

make sure we check the size of all arguments

fix a comment now that we have concurrency

make sure we check argument count everywhere

deduplicate FD extraction

Auto merge of #1298 - toc-the-younger:check-arg-count, r=RalfJung

Check that shims are called with the correct number of arguments

Fixes #1272

Some shims don't use all their arguments, I've assumed they are being called in tests with the correct number of arguments here.

Correct the test.

Address comments.

Test for too many args.

Address consistency of naming for unused/merely validated arguments.

Add a test for check_arg_count.

Check that shims are called with the correct number of arguments

Auto merge of #1392 - RalfJung:win-lock-check, r=RalfJung

Windows lock primitives: check that we are truly single-threaded

Also rustup because why not.

rename single-threaded sync test

rustup

Windows lock primitives: check that we are truly sinle-threaded

Auto merge of #1390 - RalfJung:conc, r=RalfJung

Make spin_loop yield, and a panicking thread test

@vakaras agreed it would make sense to make `spin_loop` yield, so this implements that.

Also add a test for a panic being raised in one thread, caught by the panic machinery, and then expose through the join handle. Miraculously, that just works. :)

fix setting thread name on macOS

de-duplicate <unnamed> default thread name

also test panic from thread with name

also test std::hint::spin_loop

test panics in threads

yield on x86 'pause' hint

explain what happens in a test

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

bump Rust, fix warnings

bump Rust, fix warnings

Auto merge of #1386 - RalfJung:trophy, r=RalfJung

trophy case: consistent wording (X doing Y)

Auto merge of #1383 - brson:tikv-trophy, r=RalfJung

Add another tikv stacked borrowing error to trophy case

@RalfJung please update it with whatever language is most correct :)

trophy case: consistent wording (X doing Y)

Adjust wording

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

rustup

before making a rustc PR, just to be sure

rustup

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

rustup for fixed error messages

rustup for fixed error messages

Add another tikv stacked borrowing error to trophy case

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

rustup; fix for changed error messages

rustup; fix for changed error messages

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

rustup, adjust error messages

rustup, adjust error messages

Auto merge of #1379 - RalfJung:macos-fn-name, r=RalfJung

make sure macos function has 'macos' in its name

also tiny README tweak

README tweak

make sure macos function has 'macos' in its name

Auto merge of #1284 - vakaras:add-threads-cr2, r=RalfJung

Implement basic support for concurrency (Linux/macos only)

Changes  (most new code is in `src/threads.rs` and `src/shims/foreign_items/posix.rs`):

1. Move the stack from `Machine` to a newly created `Thread` struct.
2. Add a `ThreadSet` struct that manages the threads.
3. Change `canonical_alloc_id` to create a unique allocation id for each thread local and thread (the responsible struct is `ThreadLocalStorage`)
4. Change the code to execute the thread local destructors immediately when a thread terminates.
5. Add the most basic round-robin scheduler.

This pull request depends on [these changes to the compiler](https://github.com/rust-lang/rust/pull/70598).

Fix prctl SET_NAME and GET_NAME behaviour.

Auto merge of #1377 - brson:servo-trophy, r=RalfJung

Add servo_arc to trophy case

Tweak error wording

Auto merge of #1378 - RalfJung:workspace-error, r=RalfJung

more helpful error on workspaces

more helpful error on workspaces

Add servo_arc to trophy case

Fix a regression in Windows dtors.

Use Entry API in set_dtors_running.

Improve code readability and comments.

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

rustup for more LocalDefId changes

rustup for more LocalDefId changes

Fix MacOS and Windows builds.

Delete remaining tls entries after all destructors completed.

Make From implementations non-failing.

Improve comments.

Add a test for joining in a destructor.

Merge dtors_running and last_dtor_key fields.

Add a FIXME.

Improve comments and code clarity.

Change representation and conversion of ThreadId and BlockSetId.

Use DLL_THREAD_DETACH when calling windows TLS destructor.

Make sure to remove thread local data only if we have destructor.

Improve style and comments.

Many small changes to clean up code.