Revert of [turbofan] Introduce a dedicated CheckBounds operator.

Revert of [turbofan] Introduce a dedicated CheckBounds operator. (patchset #5 id:80001 of https://codereview.chromium.org/2035893004/ )

Reason for revert:
Speculative revert since V8 roll is blocked.

Buildbot: https://build.chromium.org/p/tryserver.chromium.linux/builders/linux_chromium_chromeos_rel_ng/builds/228171

Example log: https://build.chromium.org/p/tryserver.chromium.linux/builders/linux_chromium_chromeos_rel_ng/builds/228171/steps/browser_tests%20%28with%20patch%29%20on%20Ubuntu-12.04/logs/CreateNewFolder_FileManagerBrowserTest.Test_0

Failing assert:
#
# Fatal error in ../../v8/src/compiler/node.cc, line 63
# Node::New() Error: #202:DeoptimizeUnless[1] is nullptr
#

(I take it that's a rather generic assert in TF, hence the revert is somewhat sepculative.)

Original issue's description:
> [turbofan] Introduce a dedicated CheckBounds operator.
>
> This CheckBounds simplified operator is similar to the HBoundsCheck in
> Crankshaft, and is hooked up to the new type feedback support in the
> SimplifiedLowering. We use it to check the index bounds for keyed
> property accesses.
>
> Note to perf sheriffs: This will tank quite a few benchmarks, as the
> operator makes some redundant branch elimination ineffective for
> certain patterns of keyed accesses. This does require more serious
> redundancy elimination, which we will do in a separate CL. So ignore
> any regressions from this CL, we know there will be a few.
>
> R=jarin@chromium.org
> BUG=v8:4470, v8:5100
>
> Committed: https://crrev.com/85e5567dae66a918500ae94c5568221137a0f5d4
> Cr-Commit-Position: refs/heads/master@{#36947}

TBR=jarin@chromium.org,bmeurer@chromium.org
# Skipping CQ checks because original CL landed less than 1 days ago.
NOPRESUBMIT=true
NOTREECHECKS=true
NOTRY=true
BUG=v8:4470, v8:5100
Committed: https://crrev.com/d20e8183b4317b9511390e36b296da3ee355d9d0
Cr-Commit-Position: refs/heads/master@{#36975}

[vogelheim, 2016-06-14 16:08:08]

The CQ bit was checked by vogelheim@chromium.org

[vogelheim, 2016-06-14 16:08:09]

Created Revert of [turbofan] Introduce a dedicated CheckBounds operator.

[commit-bot: I haz the power, 2016-06-14 16:08:12]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/2064163002/1

[commit-bot: I haz the power, 2016-06-14 16:08:24]

Description was changed from

==========
Revert of [turbofan] Introduce a dedicated CheckBounds operator. (patchset #5
id:80001 of https://codereview.chromium.org/2035893004/ )

Reason for revert:
Speculative revert since V8 roll is blocked.

Buildbot:
https://build.chromium.org/p/tryserver.chromium.linux/builders/linux_chromium...

Example log:
https://build.chromium.org/p/tryserver.chromium.linux/builders/linux_chromium...


Failing assert:
#
# Fatal error in ../../v8/src/compiler/node.cc, line 63
# Node::New() Error: #202:DeoptimizeUnless[1] is nullptr
#


(I take it that's a rather generic assert in TF, hence the revert is somewhat
sepculative.)

Original issue's description:
> [turbofan] Introduce a dedicated CheckBounds operator.
> 
> This CheckBounds simplified operator is similar to the HBoundsCheck in
> Crankshaft, and is hooked up to the new type feedback support in the
> SimplifiedLowering. We use it to check the index bounds for keyed
> property accesses.
> 
> Note to perf sheriffs: This will tank quite a few benchmarks, as the
> operator makes some redundant branch elimination ineffective for
> certain patterns of keyed accesses. This does require more serious
> redundancy elimination, which we will do in a separate CL. So ignore
> any regressions from this CL, we know there will be a few.
> 
> R=jarin@chromium.org
> BUG=v8:4470,v8:5100
> 
> Committed: https://crrev.com/85e5567dae66a918500ae94c5568221137a0f5d4
> Cr-Commit-Position: refs/heads/master@{#36947}

TBR=jarin@chromium.org,bmeurer@chromium.org
# Skipping CQ checks because original CL landed less than 1 days ago.
NOPRESUBMIT=true
NOTREECHECKS=true
NOTRY=true
BUG=v8:4470,v8:5100
==========

to

==========
Revert of [turbofan] Introduce a dedicated CheckBounds operator. (patchset #5
id:80001 of https://codereview.chromium.org/2035893004/ )

Reason for revert:
Speculative revert since V8 roll is blocked.

Buildbot:
https://build.chromium.org/p/tryserver.chromium.linux/builders/linux_chromium...

Example log:
https://build.chromium.org/p/tryserver.chromium.linux/builders/linux_chromium...


Failing assert:
#
# Fatal error in ../../v8/src/compiler/node.cc, line 63
# Node::New() Error: #202:DeoptimizeUnless[1] is nullptr
#


(I take it that's a rather generic assert in TF, hence the revert is somewhat
sepculative.)

Original issue's description:
> [turbofan] Introduce a dedicated CheckBounds operator.
> 
> This CheckBounds simplified operator is similar to the HBoundsCheck in
> Crankshaft, and is hooked up to the new type feedback support in the
> SimplifiedLowering. We use it to check the index bounds for keyed
> property accesses.
> 
> Note to perf sheriffs: This will tank quite a few benchmarks, as the
> operator makes some redundant branch elimination ineffective for
> certain patterns of keyed accesses. This does require more serious
> redundancy elimination, which we will do in a separate CL. So ignore
> any regressions from this CL, we know there will be a few.
> 
> R=jarin@chromium.org
> BUG=v8:4470,v8:5100
> 
> Committed: https://crrev.com/85e5567dae66a918500ae94c5568221137a0f5d4
> Cr-Commit-Position: refs/heads/master@{#36947}

TBR=jarin@chromium.org,bmeurer@chromium.org
# Skipping CQ checks because original CL landed less than 1 days ago.
NOPRESUBMIT=true
NOTREECHECKS=true
NOTRY=true
BUG=v8:4470,v8:5100
==========

[commit-bot: I haz the power, 2016-06-14 16:08:25]

Committed patchset #1 (id:1)

[commit-bot: I haz the power, 2016-06-14 16:09:45]

Description was changed from

==========
Revert of [turbofan] Introduce a dedicated CheckBounds operator. (patchset #5
id:80001 of https://codereview.chromium.org/2035893004/ )

Reason for revert:
Speculative revert since V8 roll is blocked.

Buildbot:
https://build.chromium.org/p/tryserver.chromium.linux/builders/linux_chromium...

Example log:
https://build.chromium.org/p/tryserver.chromium.linux/builders/linux_chromium...


Failing assert:
#
# Fatal error in ../../v8/src/compiler/node.cc, line 63
# Node::New() Error: #202:DeoptimizeUnless[1] is nullptr
#


(I take it that's a rather generic assert in TF, hence the revert is somewhat
sepculative.)

Original issue's description:
> [turbofan] Introduce a dedicated CheckBounds operator.
> 
> This CheckBounds simplified operator is similar to the HBoundsCheck in
> Crankshaft, and is hooked up to the new type feedback support in the
> SimplifiedLowering. We use it to check the index bounds for keyed
> property accesses.
> 
> Note to perf sheriffs: This will tank quite a few benchmarks, as the
> operator makes some redundant branch elimination ineffective for
> certain patterns of keyed accesses. This does require more serious
> redundancy elimination, which we will do in a separate CL. So ignore
> any regressions from this CL, we know there will be a few.
> 
> R=jarin@chromium.org
> BUG=v8:4470,v8:5100
> 
> Committed: https://crrev.com/85e5567dae66a918500ae94c5568221137a0f5d4
> Cr-Commit-Position: refs/heads/master@{#36947}

TBR=jarin@chromium.org,bmeurer@chromium.org
# Skipping CQ checks because original CL landed less than 1 days ago.
NOPRESUBMIT=true
NOTREECHECKS=true
NOTRY=true
BUG=v8:4470,v8:5100
==========

to

==========
Revert of [turbofan] Introduce a dedicated CheckBounds operator. (patchset #5
id:80001 of https://codereview.chromium.org/2035893004/ )

Reason for revert:
Speculative revert since V8 roll is blocked.

Buildbot:
https://build.chromium.org/p/tryserver.chromium.linux/builders/linux_chromium...

Example log:
https://build.chromium.org/p/tryserver.chromium.linux/builders/linux_chromium...

Failing assert:
#
# Fatal error in ../../v8/src/compiler/node.cc, line 63
# Node::New() Error: #202:DeoptimizeUnless[1] is nullptr
#

(I take it that's a rather generic assert in TF, hence the revert is somewhat
sepculative.)

Original issue's description:
> [turbofan] Introduce a dedicated CheckBounds operator.
>
> This CheckBounds simplified operator is similar to the HBoundsCheck in
> Crankshaft, and is hooked up to the new type feedback support in the
> SimplifiedLowering. We use it to check the index bounds for keyed
> property accesses.
>
> Note to perf sheriffs: This will tank quite a few benchmarks, as the
> operator makes some redundant branch elimination ineffective for
> certain patterns of keyed accesses. This does require more serious
> redundancy elimination, which we will do in a separate CL. So ignore
> any regressions from this CL, we know there will be a few.
>
> R=jarin@chromium.org
> BUG=v8:4470,v8:5100
>
> Committed: https://crrev.com/85e5567dae66a918500ae94c5568221137a0f5d4
> Cr-Commit-Position: refs/heads/master@{#36947}

TBR=jarin@chromium.org,bmeurer@chromium.org
# Skipping CQ checks because original CL landed less than 1 days ago.
NOPRESUBMIT=true
NOTREECHECKS=true
NOTRY=true
BUG=v8:4470,v8:5100

Committed: https://crrev.com/d20e8183b4317b9511390e36b296da3ee355d9d0
Cr-Commit-Position: refs/heads/master@{#36975}
==========

[commit-bot: I haz the power, 2016-06-14 16:09:46]

Patchset 1 (id:??) landed as
https://crrev.com/d20e8183b4317b9511390e36b296da3ee355d9d0
Cr-Commit-Position: refs/heads/master@{#36975}