From 2cc980d8a48a4341b3eb7344d3d5966cfdeea615 Mon Sep 17 00:00:00 2001 From: Michael Starzinger Date: Mon, 28 Oct 2019 13:29:00 +0100 Subject: [PATCH] [turbofan] Remove obsolete TODO in EffectControlLinearizer. R=tebbi@chromium.org Change-Id: Idd7b1de0a769b3a069e0b539cccb9c3f9820b4bd Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/1883556 Auto-Submit: Michael Starzinger Reviewed-by: Tobias Tebbi Commit-Queue: Tobias Tebbi Cr-Commit-Position: refs/heads/master@{#64586} --- src/compiler/effect-control-linearizer.cc | 2 -- 1 file changed, 2 deletions(-) diff --git a/src/compiler/effect-control-linearizer.cc b/src/compiler/effect-control-linearizer.cc index a5dfbb6b25..ae545301d1 100644 --- a/src/compiler/effect-control-linearizer.cc +++ b/src/compiler/effect-control-linearizer.cc @@ -670,8 +670,6 @@ void EffectControlLinearizer::Run() { // The frame state at block entry is determined by the frame states leaving // all predecessors. In case there is no frame state dominating this block, // we can rely on a checkpoint being present before the next deoptimization. - // TODO(mstarzinger): Eventually we will need to go hunt for a frame state - // once deoptimizing nodes roam freely through the schedule. Node* frame_state = nullptr; if (block != schedule()->start()) { // If all the predecessors have the same effect, we can use it