T0b1/wasmtime
1
0
Fork 0
Code Issues Pull Requests Packages Projects Releases Wiki Activity

Remove split_blocks_created field (#6044)

Browse Source 
This has been unused since #5731
This commit is contained in:
bjorn3
2023-03-17 17:29:39 +01:00
committed by GitHub
parent 5ebe53a351
commit 76c6ee7363
2 changed files with 1 additions and 7 deletions
Download Patch File Download Diff File Expand all files Collapse all files

3
cranelift/frontend/src/frontend.rs
View File

@@ -1095,9 +1095,6 @@ impl<'a> FunctionBuilder<'a> {
} }
fn handle_ssa_side_effects(&mut self, side_effects: SideEffects) { 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 { for modified_block in side_effects.instructions_added_to_blocks {
if self.is_pristine(modified_block) { if self.is_pristine(modified_block) {
self.func_ctx.status[modified_block] = BlockStatus::Partial; self.func_ctx.status[modified_block] = BlockStatus::Partial;

5
cranelift/frontend/src/ssa.rs
View File

@@ -63,9 +63,6 @@ pub struct SSABuilder {
/// Side effects of a `use_var` or a `seal_block` method call. /// Side effects of a `use_var` or a `seal_block` method call.
#[derive(Default)] #[derive(Default)]
pub struct SideEffects { 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<Block>,
/// When a variable is used but has never been defined before (this happens in the case of /// 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`. /// 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 /// 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 { impl SideEffects {
fn is_empty(&self) -> bool { 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()
} }
} }