-
Notifications
You must be signed in to change notification settings - Fork 13.5k
update Miri #101248
New issue
Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? # to your account
update Miri #101248
Conversation
The Miri submodule was changed cc @rust-lang/miri |
Hm... supposedly CC0 waives copyright and thus should imply the MIT+Apache that we need? |
This comment has been minimized.
This comment has been minimized.
License discussion continuing on Zulip; the short version is: it's complicated. |
The dependency in question got removed from Miri. |
@bors r+ p=1 |
☀️ Test successful - checks-actions |
Tested on commit rust-lang/rust@e4f0493. Direct link to PR: <rust-lang/rust#101248> 🎉 miri on windows: build-fail → test-pass (cc @oli-obk @RalfJung). 🎉 miri on linux: build-fail → test-pass (cc @oli-obk @RalfJung).
Finished benchmarking commit (e4f0493): comparison URL. Overall result: no relevant changes - no action needed@rustbot label: -perf-regression Instruction countThis benchmark run did not return any relevant results for this metric. Max RSS (memory usage)ResultsThis 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.
CyclesResultsThis 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.
Footnotes |
r? @ghost
Fixes #101067