Skip to content

Commit

Permalink
Rollup merge of #101976 - RalfJung:mir-semantic-phases, r=oli-obk
Browse files Browse the repository at this point in the history
MirPhase: clarify that linting is not a semantic change

r? ```@JakobDegen```
  • Loading branch information
notriddle authored Sep 20, 2022
2 parents 2c89373 + eed6fdb commit 0d2e57c
Showing 1 changed file with 4 additions and 3 deletions.
7 changes: 4 additions & 3 deletions compiler/rustc_middle/src/mir/syntax.rs
Original file line number Diff line number Diff line change
Expand Up @@ -82,9 +82,10 @@ pub enum MirPhase {
/// access to. This occurs in generator bodies. Such locals do not behave like other locals,
/// because they eg may be aliased in surprising ways. Runtime MIR has no such special locals -
/// all generator bodies are lowered and so all places that look like locals really are locals.
/// - Const prop lints: The lint pass which reports eg `200_u8 + 200_u8` as an error is run as a
/// part of analysis to runtime MIR lowering. This means that transformations which may supress
/// such errors may not run on analysis MIR.
///
/// Also note that the lint pass which reports eg `200_u8 + 200_u8` as an error is run as a part
/// of analysis to runtime MIR lowering. To ensure lints are reported reliably, this means that
/// transformations which may supress such errors should not run on analysis MIR.
Runtime(RuntimePhase),
}

Expand Down

0 comments on commit 0d2e57c

Please sign in to comment.