]> git.proxmox.com Git - rustc.git/blame - compiler/rustc_mir_transform/src/remove_noop_landing_pads.rs
New upstream version 1.64.0+dfsg1
[rustc.git] / compiler / rustc_mir_transform / src / remove_noop_landing_pads.rs
CommitLineData
c295e0f8 1use crate::MirPass;
dfeec247 2use rustc_index::bit_set::BitSet;
c295e0f8 3use rustc_middle::mir::patch::MirPatch;
ba9703b0
XL
4use rustc_middle::mir::*;
5use rustc_middle::ty::TyCtxt;
f9f354fc 6use rustc_target::spec::PanicStrategy;
ff7c6d11 7
9fa01778 8/// A pass that removes noop landing pads and replaces jumps to them with
ff7c6d11
XL
9/// `None`. This is important because otherwise LLVM generates terrible
10/// code for these.
11pub struct RemoveNoopLandingPads;
12
a2a8927a
XL
13impl<'tcx> MirPass<'tcx> for RemoveNoopLandingPads {
14 fn is_enabled(&self, sess: &rustc_session::Session) -> bool {
15 sess.panic_strategy() != PanicStrategy::Abort
2c00a5a8 16 }
2c00a5a8 17
923072b8 18 fn run_pass(&self, _tcx: TyCtxt<'tcx>, body: &mut Body<'tcx>) {
a2a8927a
XL
19 debug!("remove_noop_landing_pads({:?})", body);
20 self.remove_nop_landing_pads(body)
ff7c6d11
XL
21 }
22}
23
24impl RemoveNoopLandingPads {
8faf50e0
XL
25 fn is_nop_landing_pad(
26 &self,
27 bb: BasicBlock,
dc9dc135 28 body: &Body<'_>,
0bf4aa26 29 nop_landing_pads: &BitSet<BasicBlock>,
8faf50e0 30 ) -> bool {
dc9dc135 31 for stmt in &body[bb].statements {
e74abb32 32 match &stmt.kind {
dfeec247
XL
33 StatementKind::FakeRead(..)
34 | StatementKind::StorageLive(_)
35 | StatementKind::StorageDead(_)
36 | StatementKind::AscribeUserType(..)
3dfed10e 37 | StatementKind::Coverage(..)
dfeec247 38 | StatementKind::Nop => {
5e7ed085 39 // These are all noops in a landing pad
ff7c6d11
XL
40 }
41
29967ef6 42 StatementKind::Assign(box (place, Rvalue::Use(_) | Rvalue::Discriminant(_))) => {
e74abb32
XL
43 if place.as_local().is_some() {
44 // Writing to a local (e.g., a drop flag) does not
45 // turn a landing pad to a non-nop
46 } else {
47 return false;
48 }
ff7c6d11
XL
49 }
50
dfeec247
XL
51 StatementKind::Assign { .. }
52 | StatementKind::SetDiscriminant { .. }
04454e1e 53 | StatementKind::Deinit(..)
6a06907d 54 | StatementKind::CopyNonOverlapping(..)
dfeec247 55 | StatementKind::Retag { .. } => {
ff7c6d11
XL
56 return false;
57 }
58 }
59 }
60
dc9dc135 61 let terminator = body[bb].terminator();
ff7c6d11 62 match terminator.kind {
dfeec247
XL
63 TerminatorKind::Goto { .. }
64 | TerminatorKind::Resume
65 | TerminatorKind::SwitchInt { .. }
f035d41b 66 | TerminatorKind::FalseEdge { .. }
dfeec247 67 | TerminatorKind::FalseUnwind { .. } => {
923072b8 68 terminator.successors().all(|succ| nop_landing_pads.contains(succ))
ff7c6d11 69 }
dfeec247
XL
70 TerminatorKind::GeneratorDrop
71 | TerminatorKind::Yield { .. }
72 | TerminatorKind::Return
73 | TerminatorKind::Abort
74 | TerminatorKind::Unreachable
75 | TerminatorKind::Call { .. }
76 | TerminatorKind::Assert { .. }
77 | TerminatorKind::DropAndReplace { .. }
f9f354fc
XL
78 | TerminatorKind::Drop { .. }
79 | TerminatorKind::InlineAsm { .. } => false,
ff7c6d11
XL
80 }
81 }
82
f9f354fc 83 fn remove_nop_landing_pads(&self, body: &mut Body<'_>) {
923072b8
FG
84 debug!("body: {:#?}", body);
85
064997fb 86 // make sure there's a resume block
ff7c6d11 87 let resume_block = {
064997fb 88 let mut patch = MirPatch::new(body);
ff7c6d11 89 let resume_block = patch.resume_block();
dc9dc135 90 patch.apply(body);
ff7c6d11
XL
91 resume_block
92 };
93 debug!("remove_noop_landing_pads: resume block is {:?}", resume_block);
94
95 let mut jumps_folded = 0;
96 let mut landing_pads_removed = 0;
dc9dc135 97 let mut nop_landing_pads = BitSet::new_empty(body.basic_blocks().len());
ff7c6d11
XL
98
99 // This is a post-order traversal, so that if A post-dominates B
100 // then A will be visited before B.
dc9dc135 101 let postorder: Vec<_> = traversal::postorder(body).map(|(bb, _)| bb).collect();
ff7c6d11
XL
102 for bb in postorder {
103 debug!(" processing {:?}", bb);
1b1a35ee
XL
104 if let Some(unwind) = body[bb].terminator_mut().unwind_mut() {
105 if let Some(unwind_bb) = *unwind {
106 if nop_landing_pads.contains(unwind_bb) {
107 debug!(" removing noop landing pad");
108 landing_pads_removed += 1;
109 *unwind = None;
110 }
111 }
112 }
113
dc9dc135 114 for target in body[bb].terminator_mut().successors_mut() {
8faf50e0 115 if *target != resume_block && nop_landing_pads.contains(*target) {
ff7c6d11
XL
116 debug!(" folding noop jump to {:?} to resume block", target);
117 *target = resume_block;
118 jumps_folded += 1;
119 }
120 }
121
dc9dc135 122 let is_nop_landing_pad = self.is_nop_landing_pad(bb, body, &nop_landing_pads);
ff7c6d11 123 if is_nop_landing_pad {
8faf50e0 124 nop_landing_pads.insert(bb);
ff7c6d11
XL
125 }
126 debug!(" is_nop_landing_pad({:?}) = {}", bb, is_nop_landing_pad);
127 }
128
129 debug!("removed {:?} jumps and {:?} landing pads", jumps_folded, landing_pads_removed);
130 }
131}