diff options
| author | Nicholas Nethercote <n.nethercote@gmail.com> | 2025-08-14 11:13:24 +1000 |
|---|---|---|
| committer | Nicholas Nethercote <n.nethercote@gmail.com> | 2025-09-01 08:52:33 +1000 |
| commit | 3a0d0be586c0135bff8e8e2a280ee647337a07e7 (patch) | |
| tree | 8daa518058c5ab9f266750c1d825978c154a6842 /compiler | |
| parent | 7ea882d4c475855fe3e453f709429cf2c8be295f (diff) | |
| download | rust-3a0d0be586c0135bff8e8e2a280ee647337a07e7.tar.gz rust-3a0d0be586c0135bff8e8e2a280ee647337a07e7.zip | |
Inline and remove `dump_mir_for_pass`.
The code is more readable without it.
Diffstat (limited to 'compiler')
| -rw-r--r-- | compiler/rustc_mir_transform/src/pass_manager.rs | 22 |
1 files changed, 6 insertions, 16 deletions
diff --git a/compiler/rustc_mir_transform/src/pass_manager.rs b/compiler/rustc_mir_transform/src/pass_manager.rs index 5b3ddcc777b..37442150723 100644 --- a/compiler/rustc_mir_transform/src/pass_manager.rs +++ b/compiler/rustc_mir_transform/src/pass_manager.rs @@ -275,13 +275,14 @@ fn run_passes_inner<'tcx>( } let prof_arg = tcx.sess.prof.enabled().then(|| format!("{:?}", body.source.def_id())); + let pass_num = true; if !body.should_skip() { let validate = validate_each & tcx.sess.opts.unstable_opts.validate_mir; let lint = tcx.sess.opts.unstable_opts.lint_mir; for pass in passes { - let name = pass.name(); + let pass_name = pass.name(); if !should_run_pass(tcx, *pass, optimizations) { continue; @@ -290,7 +291,7 @@ fn run_passes_inner<'tcx>( let dump_enabled = pass.is_mir_dump_enabled(); if dump_enabled { - dump_mir_for_pass(tcx, body, name, false); + mir::dump_mir(tcx, pass_num, pass_name, &"before", body, |_, _| Ok(())); } if let Some(prof_arg) = &prof_arg { @@ -303,13 +304,13 @@ fn run_passes_inner<'tcx>( } if dump_enabled { - dump_mir_for_pass(tcx, body, name, true); + mir::dump_mir(tcx, pass_num, pass_name, &"after", body, |_, _| Ok(())); } if validate { - validate_body(tcx, body, format!("after pass {name}")); + validate_body(tcx, body, format!("after pass {pass_name}")); } if lint { - lint_body(tcx, body, format!("after pass {name}")); + lint_body(tcx, body, format!("after pass {pass_name}")); } body.pass_count += 1; @@ -345,17 +346,6 @@ pub(super) fn validate_body<'tcx>(tcx: TyCtxt<'tcx>, body: &mut Body<'tcx>, when validate::Validator { when }.run_pass(tcx, body); } -fn dump_mir_for_pass<'tcx>(tcx: TyCtxt<'tcx>, body: &Body<'tcx>, pass_name: &str, is_after: bool) { - mir::dump_mir( - tcx, - true, - pass_name, - if is_after { &"after" } else { &"before" }, - body, - |_, _| Ok(()), - ); -} - pub(super) fn dump_mir_for_phase_change<'tcx>(tcx: TyCtxt<'tcx>, body: &Body<'tcx>) { assert_eq!(body.pass_count, 0); mir::dump_mir(tcx, true, body.phase.name(), &"after", body, |_, _| Ok(())) |
