Split ConstValue into two enums#66233
Conversation
|
The job Click to expand the log.I'm a bot! I can only do what humans tell me to, so if this was not helpful or you have suggestions for improvements, please ping or otherwise contact |
src/librustc/ty/sty.rs
Outdated
|
☔ The latest upstream changes (presumably #66242) made this pull request unmergeable. Please resolve the merge conflicts. |
|
The job Click to expand the log.I'm a bot! I can only do what humans tell me to, so if this was not helpful or you have suggestions for improvements, please ping or otherwise contact |
|
I have absolutely no idea how to fix those tests. |
|
You can run the tests with |
|
The job Click to expand the log.I'm a bot! I can only do what humans tell me to, so if this was not helpful or you have suggestions for improvements, please ping or otherwise contact |
|
☔ The latest upstream changes (presumably #66129) made this pull request unmergeable. Please resolve the merge conflicts. |
|
Blessed the hashes and the mir dump. |
|
@bors r+ |
|
📌 Commit 552fa64 has been approved by |
|
☀️ Test successful - checks-azure |
|
📣 Toolstate changed by #66233! Tested on commit 5e380b7. 💔 clippy-driver on windows: test-pass → build-fail (cc @mcarton @oli-obk @Manishearth @flip1995 @yaahc @phansch @llogiq, @rust-lang/infra). |
Tested on commit rust-lang/rust@5e380b7. Direct link to PR: <rust-lang/rust#66233> 💔 clippy-driver on windows: test-pass → build-fail (cc @mcarton @oli-obk @Manishearth @flip1995 @yaahc @phansch @llogiq, @rust-lang/infra). 💔 clippy-driver on linux: test-pass → build-fail (cc @mcarton @oli-obk @Manishearth @flip1995 @yaahc @phansch @llogiq, @rust-lang/infra).
Rustup rust-lang/rust#66233 changelog: none
Hello,
Issue #59210 appeared abandoned, so I gave it a go.
Some further cleanup and refactoring may be mandated.
I did not test beyond
x.py check, since my home computer dies compiling librustc.Fixes #59210