update Miri

[RalfJung]

Fixes #65684

[rust-highfive]

r? @nikomatsakis

(rust_highfive has picked a reviewer for you, use r? to override)

[rust-highfive]

⚠️ Warning ⚠️

• These commits modify submodules.

[RalfJung]

Self-r+ because this just updates a submodule; p=1 because it fixes toolstate.

@bors r+ p=1

[bors]

📌 Commit aff0b3a3273f4c5011ec8f0d1a4324601c44c1d7 has been approved by RalfJung

[bors]

⌛ Testing commit aff0b3a3273f4c5011ec8f0d1a4324601c44c1d7 with merge 219145fcee85fcc925c37bf7c529659b18d90f5d...

[bors]

💔 Test failed - checks-azure

[RalfJung]

@oli-obk due to rust-lang/miri#1010, bootstrap now has to install xargo itself. I think that's good; it disentangles the two a bit -- bootstrap makes fewer assumptions about what cargo miri setup does.

Could you review the changes needed for that?

[oli-obk]

Heh, neat setup. I'm not sure about the install directory, but we can change it later

@bors r+

[bors]

📌 Commit 0177bc413d4f69c094a3cb7ce1240f37d43bb8e5 has been approved by oli-obk

[bors]

🌲 The tree is currently closed for pull requests below priority 1000, this pull request will be tested once the tree is reopened

[RalfJung]

The install directory is also left unchanged by this PR.

[RalfJung]

Hm, looks like we can't call cargo install this way:

2019-10-24T16:05:49.9785583Z error: Found argument '--manifest-path' which wasn't expected, or isn't valid in this context 

@bors r-

[RalfJung]

This time I build-tested it... took an hour but now it's looking good. :)

@bors r=oli-obk

[bors]

📌 Commit 1b355b6a2a30ea49f1442c0cd8a6ab3ab33fad97 has been approved by oli-obk

[bors]

⌛ Testing commit 1b355b6a2a30ea49f1442c0cd8a6ab3ab33fad97 with merge 2b6d1e0c06aba64a3de525692873dd80424c565c...

[RalfJung]

Some other PR landed in the mean time that broke Miri:

2019-10-25T20:33:57.8333743Z error[E0308]: mismatched types
2019-10-25T20:33:57.8335013Z --> src\tools\miri\src\helpers.rs:78:82
2019-10-25T20:33:57.8335166Z |
2019-10-25T20:33:57.8336103Z 78 | let place = mir::Place { base: mir::PlaceBase::Local(local), projection: Box::new([]) };
2019-10-25T20:33:57.8336416Z | ^^^^^^^^^^^^ expected reference, found struct `std::boxed::Box`
2019-10-25T20:33:57.8336985Z |
2019-10-25T20:33:57.8337152Z = note: expected type `&rustc::ty::List<rustc::mir::ProjectionElem<rustc::mir::Local, &rustc::ty::TyS<'_>>>`
2019-10-25T20:33:57.8337702Z found type `std::boxed::Box<[_; 0]>`

[RalfJung]

Rebased and fixed Miri.

@bors r=oli-obk

[bors]

📌 Commit 58da335 has been approved by oli-obk

[bors]

🌲 The tree is currently closed for pull requests below priority 1000, this pull request will be tested once the tree is reopened

[bors]

⌛ Testing commit 58da335 with merge 8318ef2...

[bors]

☀️ Test successful - checks-azure
Approved by: oli-obk
Pushing 8318ef2 to master...

[rust-highfive]

📣 Toolstate changed by #65758!

Tested on commit 8318ef2.
Direct link to PR: #65758

🎉 miri on windows: build-fail → test-pass (cc @oli-obk @eddyb @RalfJung, @rust-lang/infra).
🎉 miri on linux: build-fail → test-pass (cc @oli-obk @eddyb @RalfJung, @rust-lang/infra).