Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(91)

Unified Diff: src/compiler/bytecode-graph-builder.cc

Issue 2851723002: [turbofan] Rip out the unused OsrGuard node. (Closed)
Patch Set: Created 3 years, 8 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
« no previous file with comments | « src/compiler/ast-graph-builder.cc ('k') | src/compiler/common-operator.h » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: src/compiler/bytecode-graph-builder.cc
diff --git a/src/compiler/bytecode-graph-builder.cc b/src/compiler/bytecode-graph-builder.cc
index e4dd0e1aaf5acdfaaf1cda19a64bba9368743b48..ccc360d846e4456ccc5a7eeb6d5b58b88733eef2 100644
--- a/src/compiler/bytecode-graph-builder.cc
+++ b/src/compiler/bytecode-graph-builder.cc
@@ -318,24 +318,6 @@ void BytecodeGraphBuilder::Environment::PrepareForOsrEntry() {
if (i >= accumulator_base()) idx = Linkage::kOsrAccumulatorRegisterIndex;
values()->at(i) = graph()->NewNode(common()->OsrValue(idx), entry);
}
-
- BailoutId loop_id(builder_->bytecode_iterator().current_offset());
- Node* frame_state =
- Checkpoint(loop_id, OutputFrameStateCombine::Ignore(), false, nullptr);
- Node* checkpoint =
- graph()->NewNode(common()->Checkpoint(), frame_state, entry, entry);
- UpdateEffectDependency(checkpoint);
-
- // Create the OSR guard nodes.
- const Operator* guard_op = common()->OsrGuard(OsrGuardType::kUninitialized);
- Node* effect = checkpoint;
- for (int i = 0; i < size; i++) {
- values()->at(i) = effect =
- graph()->NewNode(guard_op, values()->at(i), effect, entry);
- }
- Node* context = effect = graph()->NewNode(guard_op, Context(), effect, entry);
- SetContext(context);
- UpdateEffectDependency(effect);
}
bool BytecodeGraphBuilder::Environment::StateValuesRequireUpdate(
« no previous file with comments | « src/compiler/ast-graph-builder.cc ('k') | src/compiler/common-operator.h » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698