From 76c6ee7363f9042a2f8ca9f9d014bdaf80499a2c Mon Sep 17 00:00:00 2001 From: bjorn3 <17426603+bjorn3@users.noreply.github.com> Date: Fri, 17 Mar 2023 17:29:39 +0100 Subject: [PATCH] Remove split_blocks_created field (#6044) This has been unused since #5731 --- cranelift/frontend/src/frontend.rs | 3 --- cranelift/frontend/src/ssa.rs | 5 +---- 2 files changed, 1 insertion(+), 7 deletions(-) diff --git a/cranelift/frontend/src/frontend.rs b/cranelift/frontend/src/frontend.rs index a6b32f6615..8c320d23a5 100644 --- a/cranelift/frontend/src/frontend.rs +++ b/cranelift/frontend/src/frontend.rs @@ -1095,9 +1095,6 @@ impl<'a> FunctionBuilder<'a> { } fn handle_ssa_side_effects(&mut self, side_effects: SideEffects) { - for split_block in side_effects.split_blocks_created { - self.func_ctx.status[split_block] = BlockStatus::Filled; - } for modified_block in side_effects.instructions_added_to_blocks { if self.is_pristine(modified_block) { self.func_ctx.status[modified_block] = BlockStatus::Partial; diff --git a/cranelift/frontend/src/ssa.rs b/cranelift/frontend/src/ssa.rs index 8e1615479a..584e6cc96e 100644 --- a/cranelift/frontend/src/ssa.rs +++ b/cranelift/frontend/src/ssa.rs @@ -63,9 +63,6 @@ pub struct SSABuilder { /// Side effects of a `use_var` or a `seal_block` method call. #[derive(Default)] pub struct SideEffects { - /// When we want to append jump arguments to a `br_table` instruction, the critical edge is - /// splitted and the newly created `Block`s are signaled here. - pub split_blocks_created: Vec, /// When a variable is used but has never been defined before (this happens in the case of /// unreachable code), a placeholder `iconst` or `fconst` value is added to the right `Block`. /// This field signals if it is the case and return the `Block` to which the initialization has @@ -75,7 +72,7 @@ pub struct SideEffects { impl SideEffects { fn is_empty(&self) -> bool { - self.split_blocks_created.is_empty() && self.instructions_added_to_blocks.is_empty() + self.instructions_added_to_blocks.is_empty() } }