-
Notifications
You must be signed in to change notification settings - Fork 13.6k
Closed
Labels
A-MIRArea: Mid-level IR (MIR) - https://blog.rust-lang.org/2016/04/19/MIR.htmlArea: Mid-level IR (MIR) - https://blog.rust-lang.org/2016/04/19/MIR.htmlC-cleanupCategory: PRs that clean code up or issues documenting cleanup.Category: PRs that clean code up or issues documenting cleanup.T-compilerRelevant to the compiler team, which will review and decide on the PR/issue.Relevant to the compiler team, which will review and decide on the PR/issue.
Description
This should be changed to return a ()
Operand::Const
(not Rvalue::Use
of that operand):
rust/src/librustc_mir_build/build/misc.rs
Lines 35 to 37 in 39b6253
crate fn unit_rvalue(&mut self) -> Rvalue<'tcx> { | |
Rvalue::Aggregate(box AggregateKind::Tuple, vec![]) | |
} |
These two other places should switch to a ()
Rvalue::Use(Operand::Const(...))
:
rust/src/librustc_mir_build/build/cfg.rs
Line 66 in 39b6253
Rvalue::Aggregate(box AggregateKind::Tuple, vec![]), |
let unit = Rvalue::Aggregate(box AggregateKind::Tuple, vec![]); |
This issue has been assigned to @lcnr via this comment.
Metadata
Metadata
Assignees
Labels
A-MIRArea: Mid-level IR (MIR) - https://blog.rust-lang.org/2016/04/19/MIR.htmlArea: Mid-level IR (MIR) - https://blog.rust-lang.org/2016/04/19/MIR.htmlC-cleanupCategory: PRs that clean code up or issues documenting cleanup.Category: PRs that clean code up or issues documenting cleanup.T-compilerRelevant to the compiler team, which will review and decide on the PR/issue.Relevant to the compiler team, which will review and decide on the PR/issue.