Revert of [turbofan] inline %StringIteratorPrototype%.next in JSBuiltinReducer.

Revert of [turbofan] inline %StringIteratorPrototype%.next in JSBuiltinReducer. (patchset #12 id:210001 of https://codereview.chromium.org/2373983004/ )

Reason for revert:
[Sheriff] Speculative revert for win dbg: https://build.chromium.org/p/client.v8/builders/V8%20Win32%20-%20debug/builds/5029

Or we have an infra problem. Manual build before seems fine:
https://build.chromium.org/p/client.v8/builders/V8%20Win32%20-%20debug/builds/5030

Original issue's description:
> [turbofan] inline %StringIteratorPrototype%.next in JSBuiltinReducer.
>
> Implement the logic for StringIterator.prototype.next in the JSBuiltinReducer in order to allow inlining when the receiver is a JS_STRING_ITERATOR_TYPE map, built ontop of the SimplifiedOperators StringCharCodeAt and the newly added StringFromCodePoint.
>
> Also introduces a new StringFromCodePoint simplified op which may be useful for other String builtins, such as String.fromCodePoint()
>
> BUG=v8:5388
> R=bmeurer@chromium.org, mstarzinger@chromium.org
>
> Committed: https://crrev.com/aed32e0f22353527993de8bceaf246fc744558f5
> Cr-Commit-Position: refs/heads/master@{#39994}

TBR=bmeurer@chromium.org,mvstanton@chromium.org,caitp@igalia.com
# Skipping CQ checks because original CL landed less than 1 days ago.
NOPRESUBMIT=true
NOTREECHECKS=true
NOTRY=true
BUG=v8:5388
Committed: https://crrev.com/4e5a4d9352e7254b349216075d8dc37bf4ffa217
Cr-Commit-Position: refs/heads/master@{#39995}

[Michael Achenbach, 2016-10-05 14:29:22]

The CQ bit was checked by machenbach@chromium.org

[Michael Achenbach, 2016-10-05 14:29:22]

Created Revert of [turbofan] inline %StringIteratorPrototype%.next in
JSBuiltinReducer.

[commit-bot: I haz the power, 2016-10-05 14:29:26]

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, 2016-10-05 14:29:40]

Description was changed from

==========
Revert of [turbofan] inline %StringIteratorPrototype%.next in JSBuiltinReducer.
(patchset #12 id:210001 of https://codereview.chromium.org/2373983004/ )

Reason for revert:
[Sheriff] Speculative revert for win dbg:
https://build.chromium.org/p/client.v8/builders/V8%20Win32%20-%20debug/builds...

Or we have an infra problem. Manual build before seems fine:
https://build.chromium.org/p/client.v8/builders/V8%20Win32%20-%20debug/builds...

Original issue's description:
> [turbofan] inline %StringIteratorPrototype%.next in JSBuiltinReducer.
> 
> Implement the logic for StringIterator.prototype.next in the JSBuiltinReducer
in order to allow inlining when the receiver is a JS_STRING_ITERATOR_TYPE map,
built ontop of the SimplifiedOperators StringCharCodeAt and the newly added
StringFromCodePoint.
> 
> Also introduces a new StringFromCodePoint simplified op which may be useful
for other String builtins, such as String.fromCodePoint()
> 
> BUG=v8:5388
> R=bmeurer@chromium.org, mstarzinger@chromium.org
> 
> Committed: https://crrev.com/aed32e0f22353527993de8bceaf246fc744558f5
> Cr-Commit-Position: refs/heads/master@{#39994}

TBR=bmeurer@chromium.org,mvstanton@chromium.org,caitp@igalia.com
# Skipping CQ checks because original CL landed less than 1 days ago.
NOPRESUBMIT=true
NOTREECHECKS=true
NOTRY=true
BUG=v8:5388
==========

to

==========
Revert of [turbofan] inline %StringIteratorPrototype%.next in JSBuiltinReducer.
(patchset #12 id:210001 of https://codereview.chromium.org/2373983004/ )

Reason for revert:
[Sheriff] Speculative revert for win dbg:
https://build.chromium.org/p/client.v8/builders/V8%20Win32%20-%20debug/builds...

Or we have an infra problem. Manual build before seems fine:
https://build.chromium.org/p/client.v8/builders/V8%20Win32%20-%20debug/builds...

Original issue's description:
> [turbofan] inline %StringIteratorPrototype%.next in JSBuiltinReducer.
> 
> Implement the logic for StringIterator.prototype.next in the JSBuiltinReducer
in order to allow inlining when the receiver is a JS_STRING_ITERATOR_TYPE map,
built ontop of the SimplifiedOperators StringCharCodeAt and the newly added
StringFromCodePoint.
> 
> Also introduces a new StringFromCodePoint simplified op which may be useful
for other String builtins, such as String.fromCodePoint()
> 
> BUG=v8:5388
> R=bmeurer@chromium.org, mstarzinger@chromium.org
> 
> Committed: https://crrev.com/aed32e0f22353527993de8bceaf246fc744558f5
> Cr-Commit-Position: refs/heads/master@{#39994}

TBR=bmeurer@chromium.org,mvstanton@chromium.org,caitp@igalia.com
# Skipping CQ checks because original CL landed less than 1 days ago.
NOPRESUBMIT=true
NOTREECHECKS=true
NOTRY=true
BUG=v8:5388
==========

[commit-bot: I haz the power, 2016-10-05 14:29:41]

Committed patchset #1 (id:1)

[commit-bot: I haz the power, 2016-10-05 14:29:56]

Description was changed from

==========
Revert of [turbofan] inline %StringIteratorPrototype%.next in JSBuiltinReducer.
(patchset #12 id:210001 of https://codereview.chromium.org/2373983004/ )

Reason for revert:
[Sheriff] Speculative revert for win dbg:
https://build.chromium.org/p/client.v8/builders/V8%20Win32%20-%20debug/builds...

Or we have an infra problem. Manual build before seems fine:
https://build.chromium.org/p/client.v8/builders/V8%20Win32%20-%20debug/builds...

Original issue's description:
> [turbofan] inline %StringIteratorPrototype%.next in JSBuiltinReducer.
> 
> Implement the logic for StringIterator.prototype.next in the JSBuiltinReducer
in order to allow inlining when the receiver is a JS_STRING_ITERATOR_TYPE map,
built ontop of the SimplifiedOperators StringCharCodeAt and the newly added
StringFromCodePoint.
> 
> Also introduces a new StringFromCodePoint simplified op which may be useful
for other String builtins, such as String.fromCodePoint()
> 
> BUG=v8:5388
> R=bmeurer@chromium.org, mstarzinger@chromium.org
> 
> Committed: https://crrev.com/aed32e0f22353527993de8bceaf246fc744558f5
> Cr-Commit-Position: refs/heads/master@{#39994}

TBR=bmeurer@chromium.org,mvstanton@chromium.org,caitp@igalia.com
# Skipping CQ checks because original CL landed less than 1 days ago.
NOPRESUBMIT=true
NOTREECHECKS=true
NOTRY=true
BUG=v8:5388
==========

to

==========
Revert of [turbofan] inline %StringIteratorPrototype%.next in JSBuiltinReducer.
(patchset #12 id:210001 of https://codereview.chromium.org/2373983004/ )

Reason for revert:
[Sheriff] Speculative revert for win dbg:
https://build.chromium.org/p/client.v8/builders/V8%20Win32%20-%20debug/builds...

Or we have an infra problem. Manual build before seems fine:
https://build.chromium.org/p/client.v8/builders/V8%20Win32%20-%20debug/builds...

Original issue's description:
> [turbofan] inline %StringIteratorPrototype%.next in JSBuiltinReducer.
>
> Implement the logic for StringIterator.prototype.next in the JSBuiltinReducer
in order to allow inlining when the receiver is a JS_STRING_ITERATOR_TYPE map,
built ontop of the SimplifiedOperators StringCharCodeAt and the newly added
StringFromCodePoint.
>
> Also introduces a new StringFromCodePoint simplified op which may be useful
for other String builtins, such as String.fromCodePoint()
>
> BUG=v8:5388
> R=bmeurer@chromium.org, mstarzinger@chromium.org
>
> Committed: https://crrev.com/aed32e0f22353527993de8bceaf246fc744558f5
> Cr-Commit-Position: refs/heads/master@{#39994}

TBR=bmeurer@chromium.org,mvstanton@chromium.org,caitp@igalia.com
# Skipping CQ checks because original CL landed less than 1 days ago.
NOPRESUBMIT=true
NOTREECHECKS=true
NOTRY=true
BUG=v8:5388

Committed: https://crrev.com/4e5a4d9352e7254b349216075d8dc37bf4ffa217
Cr-Commit-Position: refs/heads/master@{#39995}
==========

[commit-bot: I haz the power, 2016-10-05 14:29:56]

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

[caitp, 2016-10-05 15:57:09]

On 2016/10/05 14:29:56, commit-bot: I haz the power wrote:
> Patchset 1 (id:??) landed as
> https://crrev.com/4e5a4d9352e7254b349216075d8dc37bf4ffa217
> Cr-Commit-Position: refs/heads/master@{#39995}

Based on
http://build.chromium.org/p/client.v8/builders/V8%20Win32%20-%20debug/builds/...,
it looks like the String Iterator CL was not responsible for these failures. I
see that the tree has been reopened, is it safe to send a reland CL now?

[Michael Achenbach, 2016-10-05 19:52:23]

On 2016/10/05 15:57:09, caitp wrote:
> On 2016/10/05 14:29:56, commit-bot: I haz the power wrote:
> > Patchset 1 (id:??) landed as
> > https://crrev.com/4e5a4d9352e7254b349216075d8dc37bf4ffa217
> > Cr-Commit-Position: refs/heads/master@{#39995}
> 
> Based on
>
http://build.chromium.org/p/client.v8/builders/V8%20Win32%20-%20debug/builds/...,
> it looks like the String Iterator CL was not responsible for these failures. I
> see that the tree has been reopened, is it safe to send a reland CL now?

Go ahead, or I guess you already did. Sorry for the trouble.