[Interpreter] Don't try to eliminate dead-code in bytecode-array-builder

[Interpreter] Don't try to eliminate dead-code in bytecode-array-builder

Eliminating dead code in the bytecode array builder doesn't play nice
with the register elimination optimizer. We should move it to it's own
stage in the optimization pipeline, however doing so would require
refactoring of how we deal with jumps, so for now just remove the dead
code elimination optimization.

BUG=chromium:616064
Committed: https://crrev.com/2fd3f9d69b20b1a6fb4fcaae213560380fd8ef33
Cr-Commit-Position: refs/heads/master@{#36660}

[rmcilroy, 2016-06-01 15:41:10]

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

[commit-bot: I haz the power, 2016-06-01 15:41:21]

Dry run: CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/2030583002/1
View timeline at
 https://chromium-cq-status.appspot.com/patch-timeline/2030583002/1

[rmcilroy, 2016-06-01 15:41:26]

rmcilroy@chromium.org changed reviewers:
+ mythria@chromium.org

[rmcilroy, 2016-06-01 15:41:27]

Mythri, PTAL, thanks.

[commit-bot: I haz the power, 2016-06-01 15:55:20]

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

[commit-bot: I haz the power, 2016-06-01 15:55:21]

Dry run: Try jobs failed on following builders:
  v8_linux64_avx2_rel_ng on tryserver.v8 (JOB_FAILED,
http://build.chromium.org/p/tryserver.v8/builders/v8_linux64_avx2_rel_ng/buil...)
  v8_linux64_avx2_rel_ng_triggered on tryserver.v8 (JOB_FAILED,
http://build.chromium.org/p/tryserver.v8/builders/v8_linux64_avx2_rel_ng_trig...)
  v8_linux_nodcheck_rel_ng on tryserver.v8 (JOB_FAILED,
http://build.chromium.org/p/tryserver.v8/builders/v8_linux_nodcheck_rel_ng/bu...)
  v8_linux_nodcheck_rel_ng_triggered on tryserver.v8 (JOB_FAILED,
http://build.chromium.org/p/tryserver.v8/builders/v8_linux_nodcheck_rel_ng_tr...)

[mythria, 2016-06-01 15:57:34]

lgtm.

[rmcilroy, 2016-06-01 16:13:28]

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

[commit-bot: I haz the power, 2016-06-01 16:13:37]

Dry run: CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/2030583002/20001
View timeline at
 https://chromium-cq-status.appspot.com/patch-timeline/2030583002/20001

[commit-bot: I haz the power, 2016-06-01 16:45:18]

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

[commit-bot: I haz the power, 2016-06-01 16:45:19]

Dry run: Try jobs failed on following builders:
  v8_win_nosnap_shared_rel_ng on tryserver.v8 (JOB_FAILED,
http://build.chromium.org/p/tryserver.v8/builders/v8_win_nosnap_shared_rel_ng...)
  v8_win_nosnap_shared_rel_ng_triggered on tryserver.v8 (JOB_FAILED,
http://build.chromium.org/p/tryserver.v8/builders/v8_win_nosnap_shared_rel_ng...)

[rmcilroy, 2016-06-01 20:07:12]

The CQ bit was checked by rmcilroy@chromium.org

[rmcilroy, 2016-06-01 20:07:13]

The patchset sent to the CQ was uploaded after l-g-t-m from mythria@chromium.org
Link to the patchset: https://codereview.chromium.org/2030583002/#ps40001
(title: "Fix Test")

[commit-bot: I haz the power, 2016-06-01 20:07:22]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/2030583002/40001
View timeline at
 https://chromium-cq-status.appspot.com/patch-timeline/2030583002/40001

[commit-bot: I haz the power, 2016-06-01 22:14:18]

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

[commit-bot: I haz the power, 2016-06-01 22:14:19]

Try jobs failed on following builders:
  v8_linux_dbg_ng on tryserver.v8 (JOB_TIMED_OUT,
http://build.chromium.org/p/tryserver.v8/builders/v8_linux_dbg_ng/builds/6615)

[rmcilroy, 2016-06-01 22:21:28]

The CQ bit was checked by rmcilroy@chromium.org

[commit-bot: I haz the power, 2016-06-01 22:21:38]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/2030583002/40001
View timeline at
 https://chromium-cq-status.appspot.com/patch-timeline/2030583002/40001

[commit-bot: I haz the power, 2016-06-01 22:54:46]

Committed patchset #3 (id:40001)

[commit-bot: I haz the power, 2016-06-01 22:55:26]

Description was changed from

==========
[Interpreter] Don't try to eliminate dead-code in bytecode-array-builder

Eliminating dead code in the bytecode array builder doesn't play nice
with the register elimination optimizer. We should move it to it's own
stage in the optimization pipeline, however doing so would require
refactoring of how we deal with jumps, so for now just remove the dead
code elimination optimization.

BUG=chromium:616064
==========

to

==========
[Interpreter] Don't try to eliminate dead-code in bytecode-array-builder

Eliminating dead code in the bytecode array builder doesn't play nice
with the register elimination optimizer. We should move it to it's own
stage in the optimization pipeline, however doing so would require
refactoring of how we deal with jumps, so for now just remove the dead
code elimination optimization.

BUG=chromium:616064

Committed: https://crrev.com/2fd3f9d69b20b1a6fb4fcaae213560380fd8ef33
Cr-Commit-Position: refs/heads/master@{#36660}
==========

[commit-bot: I haz the power, 2016-06-01 22:55:27]

Patchset 3 (id:??) landed as
https://crrev.com/2fd3f9d69b20b1a6fb4fcaae213560380fd8ef33
Cr-Commit-Position: refs/heads/master@{#36660}