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

Ignore already sealed blocks in seal_all_blocks

Browse Source
This commit is contained in:
teapotd
2020-04-02 16:23:08 +02:00
committed by Andrew Brown
parent 654e953fbf
commit 11497a5207
2 changed files with 5 additions and 3 deletions
Download Patch File Download Diff File Expand all files Collapse all files

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

@@ -259,7 +259,7 @@ impl<'a> FunctionBuilder<'a> {
self.handle_ssa_side_effects(side_effects); self.handle_ssa_side_effects(side_effects);
} }
/// Effectively calls seal_block on all blocks in the function. /// Effectively calls seal_block on all unsealed blocks in the function.
/// ///
/// It's more efficient to seal `Block`s as soon as possible, during /// It's more efficient to seal `Block`s as soon as possible, during
/// translation, but for frontends where this is impractical to do, this /// translation, but for frontends where this is impractical to do, this

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

@@ -372,7 +372,7 @@ impl SSABuilder {
mem::replace(&mut self.side_effects, SideEffects::new()) mem::replace(&mut self.side_effects, SideEffects::new())
} }
/// Completes the global value numbering for all `Block`s in `func`. /// Completes the global value numbering for all unsealed `Block`s in `func`.
/// ///
/// It's more efficient to seal `Block`s as soon as possible, during /// It's more efficient to seal `Block`s as soon as possible, during
/// translation, but for frontends where this is impractical to do, this /// translation, but for frontends where this is impractical to do, this
@@ -383,8 +383,10 @@ impl SSABuilder {
// and creation of new blocks, however such new blocks are sealed on // and creation of new blocks, however such new blocks are sealed on
// the fly, so we don't need to account for them here. // the fly, so we don't need to account for them here.
for block in self.ssa_blocks.keys() { for block in self.ssa_blocks.keys() {
if !self.is_sealed(block) {
self.seal_one_block(block, func); self.seal_one_block(block, func);
} }
}
mem::replace(&mut self.side_effects, SideEffects::new()) mem::replace(&mut self.side_effects, SideEffects::new())
} }