[turbofan] Remove dead {is_liveness_analysis_enabled}.

[turbofan] Remove dead {is_liveness_analysis_enabled}.

R=jarin@chromium.org

Review-Url: https://codereview.chromium.org/2655533003
Cr-Commit-Position: refs/heads/master@{#42693}
Committed: https://chromium.googlesource.com/v8/v8/+/01d051ff33d9dbaa5d280a7a623e3c093c0e6bd4

[Michael Starzinger, 2017-01-24 12:14:27]

The CQ bit was checked by mstarzinger@chromium.org to run a CQ dry run

[commit-bot: I haz the power, 2017-01-24 12:14:33]

Dry run: CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[Michael Starzinger, 2017-01-24 12:15:16]

mstarzinger@chromium.org changed reviewers:
+ leszeks@chromium.org

[Michael Starzinger, 2017-01-24 12:15:16]

Jaro: PTAL.
Leszek: FYI.

[commit-bot: I haz the power, 2017-01-24 12:45:13]

The CQ bit was unchecked by commit-bot@chromium.org

[commit-bot: I haz the power, 2017-01-24 12:45:13]

Dry run: This issue passed the CQ dry run.

[Leszek Swirski, 2017-01-24 19:53:15]

I could add this functionality to the bytecode analysis if you think it's
useful?

[Michael Starzinger, 2017-01-25 09:12:11]

On 2017/01/24 19:53:15, Leszek Swirski wrote:
> I could add this functionality to the bytecode analysis if you think it's
> useful?

The functionality to disable liveness analysis is there, the flag is passed in
as a boolean into the {BytecodeAnalysis::BytecodeAnalysis}. This is just
removing dead code. Am I missing something?

[Leszek Swirski, 2017-01-25 17:33:36]

On 2017/01/25 09:12:11, Michael Starzinger wrote:
> On 2017/01/24 19:53:15, Leszek Swirski wrote:
> > I could add this functionality to the bytecode analysis if you think it's
> > useful?
> 
> The functionality to disable liveness analysis is there, the flag is passed in
> as a boolean into the {BytecodeAnalysis::BytecodeAnalysis}. This is just
> removing dead code. Am I missing something?

Argh, sorry, I was thinking of loop analysis when I wrote that. They start with
the same letter and I have a very simple hashing algorithm when skimming text :)

[Jarin, 2017-01-26 10:20:38]

lgtm

[Michael Starzinger, 2017-01-26 11:51:24]

The CQ bit was checked by mstarzinger@chromium.org

[commit-bot: I haz the power, 2017-01-26 11:51:36]

CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[commit-bot: I haz the power, 2017-01-26 12:24:11]

CQ is committing da patch.

Bot data: {"patchset_id": 1, "attempt_start_ts": 1485431484692130, "parent_rev":
"73980ae4a19d9434e3c3eb927b7b005087157e32", "commit_rev":
"01d051ff33d9dbaa5d280a7a623e3c093c0e6bd4"}

[commit-bot: I haz the power, 2017-01-26 12:24:15]

Description was changed from

==========
[turbofan] Remove dead {is_liveness_analysis_enabled}.

R=jarin@chromium.org
==========

to

==========
[turbofan] Remove dead {is_liveness_analysis_enabled}.

R=jarin@chromium.org

Review-Url: https://codereview.chromium.org/2655533003
Cr-Commit-Position: refs/heads/master@{#42693}
Committed:
https://chromium.googlesource.com/v8/v8/+/01d051ff33d9dbaa5d280a7a623e3c093c0...
==========

[commit-bot: I haz the power, 2017-01-26 12:24:15]

Committed patchset #1 (id:1) as
https://chromium.googlesource.com/v8/v8/+/01d051ff33d9dbaa5d280a7a623e3c093c0...