New issue
Advanced search Search tips

Issue 879898 link

Starred by 1 user

Issue metadata

Status: Verified
Owner:
Closed: Sep 3
Cc:
Components:
EstimatedDays: ----
NextAction: ----
OS: Linux
Pri: 1
Type: Bug-Security



Sign in to add a comment

CHECK failure: TypeError: node #28:JSToNumber type Numeric is not Number in verifier.cc

Project Member Reported by ClusterFuzz, Sep 2

Issue description

Detailed report: https://clusterfuzz.com/testcase?key=5085453708165120

Fuzzer: ochang_js_fuzzer
Job Type: linux_asan_d8_dbg
Platform Id: linux

Crash Type: CHECK failure
Crash Address: 
Crash State:
  TypeError: node #28:JSToNumber type Numeric is not Number in verifier.cc
  v8::internal::compiler::Verifier::Visitor::CheckTypeIs
  v8::internal::compiler::Verifier::Visitor::Check
  
Sanitizer: address (ASAN)

Regressed: https://clusterfuzz.com/revisions?job=linux_asan_d8_dbg&range=55555:55556

Reproducer Testcase: https://clusterfuzz.com/download?testcase_id=5085453708165120

Issue filed automatically.

See https://github.com/google/clusterfuzz-tools for more information.
 
Project Member

Comment 1 by ClusterFuzz, Sep 2

Components: Blink>JavaScript>Compiler
Labels: Test-Predator-Auto-Components
Automatically applying components based on crash stacktrace and information from OWNERS files.

If this is incorrect, please apply the Test-Predator-Wrong-Components label.
Project Member

Comment 2 by ClusterFuzz, Sep 2

Labels: Test-Predator-Auto-Owner
Owner: bmeu...@chromium.org
Status: Assigned (was: Untriaged)
Automatically assigning owner based on suspected regression changelist https://chromium.googlesource.com/v8/v8/+/e034c1ad9c44b146cd9b4fed54a900b2e4845976 ([turbofan] ToNumeric(x) does ToNumber(x) for all non-BigInt primitives.).

If this is incorrect, please let us know why and apply the Test-Predator-Wrong-CLs label. If you aren't the correct owner for this issue, please unassign yourself as soon as possible so it can be re-triaged.
Project Member

Comment 3 by sheriffbot@chromium.org, Sep 2

Labels: Pri-1
Cc: neis@chromium.org
Status: Started (was: Assigned)
Project Member

Comment 5 by bugdroid1@chromium.org, Sep 3

The following revision refers to this bug:
  https://chromium.googlesource.com/v8/v8.git/+/b89811227717381e7d50d03cf19fca00f39cc3aa

commit b89811227717381e7d50d03cf19fca00f39cc3aa
Author: Benedikt Meurer <bmeurer@chromium.org>
Date: Mon Sep 03 19:14:09 2018

[turbofan] Improve typing of ToNumeric and ToNumber.

The previous typing rules for ToNumeric and ToNumber didn't match on the
NonBigIntPrimitive input set, which causes trouble when we morph ToNumeric
nodes into ToNumber nodes, and generally lead to worse typings in the
graph, and thus worse code generation. This change improves the existing
typing rules and turns ToNumber into a chokepoint again.

Bug:  chromium:879898 , v8:8015
Change-Id: I4a7ff0e9c420c5dcfdb2b96884e019a5943828a4
Reviewed-on: https://chromium-review.googlesource.com/1201522
Reviewed-by: Georg Neis <neis@chromium.org>
Commit-Queue: Benedikt Meurer <bmeurer@chromium.org>
Cr-Commit-Position: refs/heads/master@{#55595}
[modify] https://crrev.com/b89811227717381e7d50d03cf19fca00f39cc3aa/src/compiler/operation-typer.cc
[modify] https://crrev.com/b89811227717381e7d50d03cf19fca00f39cc3aa/src/compiler/operation-typer.h
[modify] https://crrev.com/b89811227717381e7d50d03cf19fca00f39cc3aa/src/compiler/types.h
[add] https://crrev.com/b89811227717381e7d50d03cf19fca00f39cc3aa/test/mjsunit/regress/regress-crbug-879898.js

Status: Fixed (was: Started)
Project Member

Comment 7 by bugdroid1@chromium.org, Sep 3

The following revision refers to this bug:
  https://chromium.googlesource.com/v8/v8.git/+/b89811227717381e7d50d03cf19fca00f39cc3aa

commit b89811227717381e7d50d03cf19fca00f39cc3aa
Author: Benedikt Meurer <bmeurer@chromium.org>
Date: Mon Sep 03 19:14:09 2018

[turbofan] Improve typing of ToNumeric and ToNumber.

The previous typing rules for ToNumeric and ToNumber didn't match on the
NonBigIntPrimitive input set, which causes trouble when we morph ToNumeric
nodes into ToNumber nodes, and generally lead to worse typings in the
graph, and thus worse code generation. This change improves the existing
typing rules and turns ToNumber into a chokepoint again.

Bug:  chromium:879898 , v8:8015
Change-Id: I4a7ff0e9c420c5dcfdb2b96884e019a5943828a4
Reviewed-on: https://chromium-review.googlesource.com/1201522
Reviewed-by: Georg Neis <neis@chromium.org>
Commit-Queue: Benedikt Meurer <bmeurer@chromium.org>
Cr-Commit-Position: refs/heads/master@{#55595}
[modify] https://crrev.com/b89811227717381e7d50d03cf19fca00f39cc3aa/src/compiler/operation-typer.cc
[modify] https://crrev.com/b89811227717381e7d50d03cf19fca00f39cc3aa/src/compiler/operation-typer.h
[modify] https://crrev.com/b89811227717381e7d50d03cf19fca00f39cc3aa/src/compiler/types.h
[add] https://crrev.com/b89811227717381e7d50d03cf19fca00f39cc3aa/test/mjsunit/regress/regress-crbug-879898.js

Project Member

Comment 8 by bugdroid1@chromium.org, Sep 3

The following revision refers to this bug:
  https://chromium.googlesource.com/v8/v8.git/+/b89811227717381e7d50d03cf19fca00f39cc3aa

commit b89811227717381e7d50d03cf19fca00f39cc3aa
Author: Benedikt Meurer <bmeurer@chromium.org>
Date: Mon Sep 03 19:14:09 2018

[turbofan] Improve typing of ToNumeric and ToNumber.

The previous typing rules for ToNumeric and ToNumber didn't match on the
NonBigIntPrimitive input set, which causes trouble when we morph ToNumeric
nodes into ToNumber nodes, and generally lead to worse typings in the
graph, and thus worse code generation. This change improves the existing
typing rules and turns ToNumber into a chokepoint again.

Bug:  chromium:879898 , v8:8015
Change-Id: I4a7ff0e9c420c5dcfdb2b96884e019a5943828a4
Reviewed-on: https://chromium-review.googlesource.com/1201522
Reviewed-by: Georg Neis <neis@chromium.org>
Commit-Queue: Benedikt Meurer <bmeurer@chromium.org>
Cr-Commit-Position: refs/heads/master@{#55595}
[modify] https://crrev.com/b89811227717381e7d50d03cf19fca00f39cc3aa/src/compiler/operation-typer.cc
[modify] https://crrev.com/b89811227717381e7d50d03cf19fca00f39cc3aa/src/compiler/operation-typer.h
[modify] https://crrev.com/b89811227717381e7d50d03cf19fca00f39cc3aa/src/compiler/types.h
[add] https://crrev.com/b89811227717381e7d50d03cf19fca00f39cc3aa/test/mjsunit/regress/regress-crbug-879898.js

Project Member

Comment 9 by bugdroid1@chromium.org, Sep 3

The following revision refers to this bug:
  https://chromium.googlesource.com/v8/v8.git/+/b89811227717381e7d50d03cf19fca00f39cc3aa

commit b89811227717381e7d50d03cf19fca00f39cc3aa
Author: Benedikt Meurer <bmeurer@chromium.org>
Date: Mon Sep 03 19:14:09 2018

[turbofan] Improve typing of ToNumeric and ToNumber.

The previous typing rules for ToNumeric and ToNumber didn't match on the
NonBigIntPrimitive input set, which causes trouble when we morph ToNumeric
nodes into ToNumber nodes, and generally lead to worse typings in the
graph, and thus worse code generation. This change improves the existing
typing rules and turns ToNumber into a chokepoint again.

Bug:  chromium:879898 , v8:8015
Change-Id: I4a7ff0e9c420c5dcfdb2b96884e019a5943828a4
Reviewed-on: https://chromium-review.googlesource.com/1201522
Reviewed-by: Georg Neis <neis@chromium.org>
Commit-Queue: Benedikt Meurer <bmeurer@chromium.org>
Cr-Commit-Position: refs/heads/master@{#55595}
[modify] https://crrev.com/b89811227717381e7d50d03cf19fca00f39cc3aa/src/compiler/operation-typer.cc
[modify] https://crrev.com/b89811227717381e7d50d03cf19fca00f39cc3aa/src/compiler/operation-typer.h
[modify] https://crrev.com/b89811227717381e7d50d03cf19fca00f39cc3aa/src/compiler/types.h
[add] https://crrev.com/b89811227717381e7d50d03cf19fca00f39cc3aa/test/mjsunit/regress/regress-crbug-879898.js

Project Member

Comment 10 by bugdroid1@chromium.org, Sep 3

The following revision refers to this bug:
  https://chromium.googlesource.com/v8/v8.git/+/b89811227717381e7d50d03cf19fca00f39cc3aa

commit b89811227717381e7d50d03cf19fca00f39cc3aa
Author: Benedikt Meurer <bmeurer@chromium.org>
Date: Mon Sep 03 19:14:09 2018

[turbofan] Improve typing of ToNumeric and ToNumber.

The previous typing rules for ToNumeric and ToNumber didn't match on the
NonBigIntPrimitive input set, which causes trouble when we morph ToNumeric
nodes into ToNumber nodes, and generally lead to worse typings in the
graph, and thus worse code generation. This change improves the existing
typing rules and turns ToNumber into a chokepoint again.

Bug:  chromium:879898 , v8:8015
Change-Id: I4a7ff0e9c420c5dcfdb2b96884e019a5943828a4
Reviewed-on: https://chromium-review.googlesource.com/1201522
Reviewed-by: Georg Neis <neis@chromium.org>
Commit-Queue: Benedikt Meurer <bmeurer@chromium.org>
Cr-Commit-Position: refs/heads/master@{#55595}
[modify] https://crrev.com/b89811227717381e7d50d03cf19fca00f39cc3aa/src/compiler/operation-typer.cc
[modify] https://crrev.com/b89811227717381e7d50d03cf19fca00f39cc3aa/src/compiler/operation-typer.h
[modify] https://crrev.com/b89811227717381e7d50d03cf19fca00f39cc3aa/src/compiler/types.h
[add] https://crrev.com/b89811227717381e7d50d03cf19fca00f39cc3aa/test/mjsunit/regress/regress-crbug-879898.js

Project Member

Comment 11 by bugdroid1@chromium.org, Sep 3

The following revision refers to this bug:
  https://chromium.googlesource.com/v8/v8.git/+/b89811227717381e7d50d03cf19fca00f39cc3aa

commit b89811227717381e7d50d03cf19fca00f39cc3aa
Author: Benedikt Meurer <bmeurer@chromium.org>
Date: Mon Sep 03 19:14:09 2018

[turbofan] Improve typing of ToNumeric and ToNumber.

The previous typing rules for ToNumeric and ToNumber didn't match on the
NonBigIntPrimitive input set, which causes trouble when we morph ToNumeric
nodes into ToNumber nodes, and generally lead to worse typings in the
graph, and thus worse code generation. This change improves the existing
typing rules and turns ToNumber into a chokepoint again.

Bug:  chromium:879898 , v8:8015
Change-Id: I4a7ff0e9c420c5dcfdb2b96884e019a5943828a4
Reviewed-on: https://chromium-review.googlesource.com/1201522
Reviewed-by: Georg Neis <neis@chromium.org>
Commit-Queue: Benedikt Meurer <bmeurer@chromium.org>
Cr-Commit-Position: refs/heads/master@{#55595}
[modify] https://crrev.com/b89811227717381e7d50d03cf19fca00f39cc3aa/src/compiler/operation-typer.cc
[modify] https://crrev.com/b89811227717381e7d50d03cf19fca00f39cc3aa/src/compiler/operation-typer.h
[modify] https://crrev.com/b89811227717381e7d50d03cf19fca00f39cc3aa/src/compiler/types.h
[add] https://crrev.com/b89811227717381e7d50d03cf19fca00f39cc3aa/test/mjsunit/regress/regress-crbug-879898.js

Project Member

Comment 12 by bugdroid1@chromium.org, Sep 3

The following revision refers to this bug:
  https://chromium.googlesource.com/v8/v8.git/+/b89811227717381e7d50d03cf19fca00f39cc3aa

commit b89811227717381e7d50d03cf19fca00f39cc3aa
Author: Benedikt Meurer <bmeurer@chromium.org>
Date: Mon Sep 03 19:14:09 2018

[turbofan] Improve typing of ToNumeric and ToNumber.

The previous typing rules for ToNumeric and ToNumber didn't match on the
NonBigIntPrimitive input set, which causes trouble when we morph ToNumeric
nodes into ToNumber nodes, and generally lead to worse typings in the
graph, and thus worse code generation. This change improves the existing
typing rules and turns ToNumber into a chokepoint again.

Bug:  chromium:879898 , v8:8015
Change-Id: I4a7ff0e9c420c5dcfdb2b96884e019a5943828a4
Reviewed-on: https://chromium-review.googlesource.com/1201522
Reviewed-by: Georg Neis <neis@chromium.org>
Commit-Queue: Benedikt Meurer <bmeurer@chromium.org>
Cr-Commit-Position: refs/heads/master@{#55595}
[modify] https://crrev.com/b89811227717381e7d50d03cf19fca00f39cc3aa/src/compiler/operation-typer.cc
[modify] https://crrev.com/b89811227717381e7d50d03cf19fca00f39cc3aa/src/compiler/operation-typer.h
[modify] https://crrev.com/b89811227717381e7d50d03cf19fca00f39cc3aa/src/compiler/types.h
[add] https://crrev.com/b89811227717381e7d50d03cf19fca00f39cc3aa/test/mjsunit/regress/regress-crbug-879898.js

Project Member

Comment 13 by bugdroid1@chromium.org, Sep 3

The following revision refers to this bug:
  https://chromium.googlesource.com/v8/v8.git/+/b89811227717381e7d50d03cf19fca00f39cc3aa

commit b89811227717381e7d50d03cf19fca00f39cc3aa
Author: Benedikt Meurer <bmeurer@chromium.org>
Date: Mon Sep 03 19:14:09 2018

[turbofan] Improve typing of ToNumeric and ToNumber.

The previous typing rules for ToNumeric and ToNumber didn't match on the
NonBigIntPrimitive input set, which causes trouble when we morph ToNumeric
nodes into ToNumber nodes, and generally lead to worse typings in the
graph, and thus worse code generation. This change improves the existing
typing rules and turns ToNumber into a chokepoint again.

Bug:  chromium:879898 , v8:8015
Change-Id: I4a7ff0e9c420c5dcfdb2b96884e019a5943828a4
Reviewed-on: https://chromium-review.googlesource.com/1201522
Reviewed-by: Georg Neis <neis@chromium.org>
Commit-Queue: Benedikt Meurer <bmeurer@chromium.org>
Cr-Commit-Position: refs/heads/master@{#55595}
[modify] https://crrev.com/b89811227717381e7d50d03cf19fca00f39cc3aa/src/compiler/operation-typer.cc
[modify] https://crrev.com/b89811227717381e7d50d03cf19fca00f39cc3aa/src/compiler/operation-typer.h
[modify] https://crrev.com/b89811227717381e7d50d03cf19fca00f39cc3aa/src/compiler/types.h
[add] https://crrev.com/b89811227717381e7d50d03cf19fca00f39cc3aa/test/mjsunit/regress/regress-crbug-879898.js

Project Member

Comment 14 by bugdroid1@chromium.org, Sep 3

The following revision refers to this bug:
  https://chromium.googlesource.com/v8/v8.git/+/b89811227717381e7d50d03cf19fca00f39cc3aa

commit b89811227717381e7d50d03cf19fca00f39cc3aa
Author: Benedikt Meurer <bmeurer@chromium.org>
Date: Mon Sep 03 19:14:09 2018

[turbofan] Improve typing of ToNumeric and ToNumber.

The previous typing rules for ToNumeric and ToNumber didn't match on the
NonBigIntPrimitive input set, which causes trouble when we morph ToNumeric
nodes into ToNumber nodes, and generally lead to worse typings in the
graph, and thus worse code generation. This change improves the existing
typing rules and turns ToNumber into a chokepoint again.

Bug:  chromium:879898 , v8:8015
Change-Id: I4a7ff0e9c420c5dcfdb2b96884e019a5943828a4
Reviewed-on: https://chromium-review.googlesource.com/1201522
Reviewed-by: Georg Neis <neis@chromium.org>
Commit-Queue: Benedikt Meurer <bmeurer@chromium.org>
Cr-Commit-Position: refs/heads/master@{#55595}
[modify] https://crrev.com/b89811227717381e7d50d03cf19fca00f39cc3aa/src/compiler/operation-typer.cc
[modify] https://crrev.com/b89811227717381e7d50d03cf19fca00f39cc3aa/src/compiler/operation-typer.h
[modify] https://crrev.com/b89811227717381e7d50d03cf19fca00f39cc3aa/src/compiler/types.h
[add] https://crrev.com/b89811227717381e7d50d03cf19fca00f39cc3aa/test/mjsunit/regress/regress-crbug-879898.js

Project Member

Comment 15 by bugdroid1@chromium.org, Sep 3

The following revision refers to this bug:
  https://chromium.googlesource.com/v8/v8.git/+/b89811227717381e7d50d03cf19fca00f39cc3aa

commit b89811227717381e7d50d03cf19fca00f39cc3aa
Author: Benedikt Meurer <bmeurer@chromium.org>
Date: Mon Sep 03 19:14:09 2018

[turbofan] Improve typing of ToNumeric and ToNumber.

The previous typing rules for ToNumeric and ToNumber didn't match on the
NonBigIntPrimitive input set, which causes trouble when we morph ToNumeric
nodes into ToNumber nodes, and generally lead to worse typings in the
graph, and thus worse code generation. This change improves the existing
typing rules and turns ToNumber into a chokepoint again.

Bug:  chromium:879898 , v8:8015
Change-Id: I4a7ff0e9c420c5dcfdb2b96884e019a5943828a4
Reviewed-on: https://chromium-review.googlesource.com/1201522
Reviewed-by: Georg Neis <neis@chromium.org>
Commit-Queue: Benedikt Meurer <bmeurer@chromium.org>
Cr-Commit-Position: refs/heads/master@{#55595}
[modify] https://crrev.com/b89811227717381e7d50d03cf19fca00f39cc3aa/src/compiler/operation-typer.cc
[modify] https://crrev.com/b89811227717381e7d50d03cf19fca00f39cc3aa/src/compiler/operation-typer.h
[modify] https://crrev.com/b89811227717381e7d50d03cf19fca00f39cc3aa/src/compiler/types.h
[add] https://crrev.com/b89811227717381e7d50d03cf19fca00f39cc3aa/test/mjsunit/regress/regress-crbug-879898.js

Project Member

Comment 16 by bugdroid1@chromium.org, Sep 3

The following revision refers to this bug:
  https://chromium.googlesource.com/v8/v8.git/+/b89811227717381e7d50d03cf19fca00f39cc3aa

commit b89811227717381e7d50d03cf19fca00f39cc3aa
Author: Benedikt Meurer <bmeurer@chromium.org>
Date: Mon Sep 03 19:14:09 2018

[turbofan] Improve typing of ToNumeric and ToNumber.

The previous typing rules for ToNumeric and ToNumber didn't match on the
NonBigIntPrimitive input set, which causes trouble when we morph ToNumeric
nodes into ToNumber nodes, and generally lead to worse typings in the
graph, and thus worse code generation. This change improves the existing
typing rules and turns ToNumber into a chokepoint again.

Bug:  chromium:879898 , v8:8015
Change-Id: I4a7ff0e9c420c5dcfdb2b96884e019a5943828a4
Reviewed-on: https://chromium-review.googlesource.com/1201522
Reviewed-by: Georg Neis <neis@chromium.org>
Commit-Queue: Benedikt Meurer <bmeurer@chromium.org>
Cr-Commit-Position: refs/heads/master@{#55595}
[modify] https://crrev.com/b89811227717381e7d50d03cf19fca00f39cc3aa/src/compiler/operation-typer.cc
[modify] https://crrev.com/b89811227717381e7d50d03cf19fca00f39cc3aa/src/compiler/operation-typer.h
[modify] https://crrev.com/b89811227717381e7d50d03cf19fca00f39cc3aa/src/compiler/types.h
[add] https://crrev.com/b89811227717381e7d50d03cf19fca00f39cc3aa/test/mjsunit/regress/regress-crbug-879898.js

Project Member

Comment 17 by bugdroid1@chromium.org, Sep 3

The following revision refers to this bug:
  https://chromium.googlesource.com/v8/v8.git/+/b89811227717381e7d50d03cf19fca00f39cc3aa

commit b89811227717381e7d50d03cf19fca00f39cc3aa
Author: Benedikt Meurer <bmeurer@chromium.org>
Date: Mon Sep 03 19:14:09 2018

[turbofan] Improve typing of ToNumeric and ToNumber.

The previous typing rules for ToNumeric and ToNumber didn't match on the
NonBigIntPrimitive input set, which causes trouble when we morph ToNumeric
nodes into ToNumber nodes, and generally lead to worse typings in the
graph, and thus worse code generation. This change improves the existing
typing rules and turns ToNumber into a chokepoint again.

Bug:  chromium:879898 , v8:8015
Change-Id: I4a7ff0e9c420c5dcfdb2b96884e019a5943828a4
Reviewed-on: https://chromium-review.googlesource.com/1201522
Reviewed-by: Georg Neis <neis@chromium.org>
Commit-Queue: Benedikt Meurer <bmeurer@chromium.org>
Cr-Commit-Position: refs/heads/master@{#55595}
[modify] https://crrev.com/b89811227717381e7d50d03cf19fca00f39cc3aa/src/compiler/operation-typer.cc
[modify] https://crrev.com/b89811227717381e7d50d03cf19fca00f39cc3aa/src/compiler/operation-typer.h
[modify] https://crrev.com/b89811227717381e7d50d03cf19fca00f39cc3aa/src/compiler/types.h
[add] https://crrev.com/b89811227717381e7d50d03cf19fca00f39cc3aa/test/mjsunit/regress/regress-crbug-879898.js

Project Member

Comment 18 by bugdroid1@chromium.org, Sep 3

The following revision refers to this bug:
  https://chromium.googlesource.com/v8/v8.git/+/b89811227717381e7d50d03cf19fca00f39cc3aa

commit b89811227717381e7d50d03cf19fca00f39cc3aa
Author: Benedikt Meurer <bmeurer@chromium.org>
Date: Mon Sep 03 19:14:09 2018

[turbofan] Improve typing of ToNumeric and ToNumber.

The previous typing rules for ToNumeric and ToNumber didn't match on the
NonBigIntPrimitive input set, which causes trouble when we morph ToNumeric
nodes into ToNumber nodes, and generally lead to worse typings in the
graph, and thus worse code generation. This change improves the existing
typing rules and turns ToNumber into a chokepoint again.

Bug:  chromium:879898 , v8:8015
Change-Id: I4a7ff0e9c420c5dcfdb2b96884e019a5943828a4
Reviewed-on: https://chromium-review.googlesource.com/1201522
Reviewed-by: Georg Neis <neis@chromium.org>
Commit-Queue: Benedikt Meurer <bmeurer@chromium.org>
Cr-Commit-Position: refs/heads/master@{#55595}
[modify] https://crrev.com/b89811227717381e7d50d03cf19fca00f39cc3aa/src/compiler/operation-typer.cc
[modify] https://crrev.com/b89811227717381e7d50d03cf19fca00f39cc3aa/src/compiler/operation-typer.h
[modify] https://crrev.com/b89811227717381e7d50d03cf19fca00f39cc3aa/src/compiler/types.h
[add] https://crrev.com/b89811227717381e7d50d03cf19fca00f39cc3aa/test/mjsunit/regress/regress-crbug-879898.js

Project Member

Comment 19 by bugdroid1@chromium.org, Sep 3

The following revision refers to this bug:
  https://chromium.googlesource.com/v8/v8.git/+/b89811227717381e7d50d03cf19fca00f39cc3aa

commit b89811227717381e7d50d03cf19fca00f39cc3aa
Author: Benedikt Meurer <bmeurer@chromium.org>
Date: Mon Sep 03 19:14:09 2018

[turbofan] Improve typing of ToNumeric and ToNumber.

The previous typing rules for ToNumeric and ToNumber didn't match on the
NonBigIntPrimitive input set, which causes trouble when we morph ToNumeric
nodes into ToNumber nodes, and generally lead to worse typings in the
graph, and thus worse code generation. This change improves the existing
typing rules and turns ToNumber into a chokepoint again.

Bug:  chromium:879898 , v8:8015
Change-Id: I4a7ff0e9c420c5dcfdb2b96884e019a5943828a4
Reviewed-on: https://chromium-review.googlesource.com/1201522
Reviewed-by: Georg Neis <neis@chromium.org>
Commit-Queue: Benedikt Meurer <bmeurer@chromium.org>
Cr-Commit-Position: refs/heads/master@{#55595}
[modify] https://crrev.com/b89811227717381e7d50d03cf19fca00f39cc3aa/src/compiler/operation-typer.cc
[modify] https://crrev.com/b89811227717381e7d50d03cf19fca00f39cc3aa/src/compiler/operation-typer.h
[modify] https://crrev.com/b89811227717381e7d50d03cf19fca00f39cc3aa/src/compiler/types.h
[add] https://crrev.com/b89811227717381e7d50d03cf19fca00f39cc3aa/test/mjsunit/regress/regress-crbug-879898.js

Project Member

Comment 20 by bugdroid1@chromium.org, Sep 3

The following revision refers to this bug:
  https://chromium.googlesource.com/v8/v8.git/+/b89811227717381e7d50d03cf19fca00f39cc3aa

commit b89811227717381e7d50d03cf19fca00f39cc3aa
Author: Benedikt Meurer <bmeurer@chromium.org>
Date: Mon Sep 03 19:14:09 2018

[turbofan] Improve typing of ToNumeric and ToNumber.

The previous typing rules for ToNumeric and ToNumber didn't match on the
NonBigIntPrimitive input set, which causes trouble when we morph ToNumeric
nodes into ToNumber nodes, and generally lead to worse typings in the
graph, and thus worse code generation. This change improves the existing
typing rules and turns ToNumber into a chokepoint again.

Bug:  chromium:879898 , v8:8015
Change-Id: I4a7ff0e9c420c5dcfdb2b96884e019a5943828a4
Reviewed-on: https://chromium-review.googlesource.com/1201522
Reviewed-by: Georg Neis <neis@chromium.org>
Commit-Queue: Benedikt Meurer <bmeurer@chromium.org>
Cr-Commit-Position: refs/heads/master@{#55595}
[modify] https://crrev.com/b89811227717381e7d50d03cf19fca00f39cc3aa/src/compiler/operation-typer.cc
[modify] https://crrev.com/b89811227717381e7d50d03cf19fca00f39cc3aa/src/compiler/operation-typer.h
[modify] https://crrev.com/b89811227717381e7d50d03cf19fca00f39cc3aa/src/compiler/types.h
[add] https://crrev.com/b89811227717381e7d50d03cf19fca00f39cc3aa/test/mjsunit/regress/regress-crbug-879898.js

Project Member

Comment 21 by bugdroid1@chromium.org, Sep 3

The following revision refers to this bug:
  https://chromium.googlesource.com/v8/v8.git/+/b89811227717381e7d50d03cf19fca00f39cc3aa

commit b89811227717381e7d50d03cf19fca00f39cc3aa
Author: Benedikt Meurer <bmeurer@chromium.org>
Date: Mon Sep 03 19:14:09 2018

[turbofan] Improve typing of ToNumeric and ToNumber.

The previous typing rules for ToNumeric and ToNumber didn't match on the
NonBigIntPrimitive input set, which causes trouble when we morph ToNumeric
nodes into ToNumber nodes, and generally lead to worse typings in the
graph, and thus worse code generation. This change improves the existing
typing rules and turns ToNumber into a chokepoint again.

Bug:  chromium:879898 , v8:8015
Change-Id: I4a7ff0e9c420c5dcfdb2b96884e019a5943828a4
Reviewed-on: https://chromium-review.googlesource.com/1201522
Reviewed-by: Georg Neis <neis@chromium.org>
Commit-Queue: Benedikt Meurer <bmeurer@chromium.org>
Cr-Commit-Position: refs/heads/master@{#55595}
[modify] https://crrev.com/b89811227717381e7d50d03cf19fca00f39cc3aa/src/compiler/operation-typer.cc
[modify] https://crrev.com/b89811227717381e7d50d03cf19fca00f39cc3aa/src/compiler/operation-typer.h
[modify] https://crrev.com/b89811227717381e7d50d03cf19fca00f39cc3aa/src/compiler/types.h
[add] https://crrev.com/b89811227717381e7d50d03cf19fca00f39cc3aa/test/mjsunit/regress/regress-crbug-879898.js

Project Member

Comment 22 by bugdroid1@chromium.org, Sep 3

The following revision refers to this bug:
  https://chromium.googlesource.com/v8/v8.git/+/b89811227717381e7d50d03cf19fca00f39cc3aa

commit b89811227717381e7d50d03cf19fca00f39cc3aa
Author: Benedikt Meurer <bmeurer@chromium.org>
Date: Mon Sep 03 19:14:09 2018

[turbofan] Improve typing of ToNumeric and ToNumber.

The previous typing rules for ToNumeric and ToNumber didn't match on the
NonBigIntPrimitive input set, which causes trouble when we morph ToNumeric
nodes into ToNumber nodes, and generally lead to worse typings in the
graph, and thus worse code generation. This change improves the existing
typing rules and turns ToNumber into a chokepoint again.

Bug:  chromium:879898 , v8:8015
Change-Id: I4a7ff0e9c420c5dcfdb2b96884e019a5943828a4
Reviewed-on: https://chromium-review.googlesource.com/1201522
Reviewed-by: Georg Neis <neis@chromium.org>
Commit-Queue: Benedikt Meurer <bmeurer@chromium.org>
Cr-Commit-Position: refs/heads/master@{#55595}
[modify] https://crrev.com/b89811227717381e7d50d03cf19fca00f39cc3aa/src/compiler/operation-typer.cc
[modify] https://crrev.com/b89811227717381e7d50d03cf19fca00f39cc3aa/src/compiler/operation-typer.h
[modify] https://crrev.com/b89811227717381e7d50d03cf19fca00f39cc3aa/src/compiler/types.h
[add] https://crrev.com/b89811227717381e7d50d03cf19fca00f39cc3aa/test/mjsunit/regress/regress-crbug-879898.js

Project Member

Comment 23 by bugdroid1@chromium.org, Sep 3

The following revision refers to this bug:
  https://chromium.googlesource.com/v8/v8.git/+/b89811227717381e7d50d03cf19fca00f39cc3aa

commit b89811227717381e7d50d03cf19fca00f39cc3aa
Author: Benedikt Meurer <bmeurer@chromium.org>
Date: Mon Sep 03 19:14:09 2018

[turbofan] Improve typing of ToNumeric and ToNumber.

The previous typing rules for ToNumeric and ToNumber didn't match on the
NonBigIntPrimitive input set, which causes trouble when we morph ToNumeric
nodes into ToNumber nodes, and generally lead to worse typings in the
graph, and thus worse code generation. This change improves the existing
typing rules and turns ToNumber into a chokepoint again.

Bug:  chromium:879898 , v8:8015
Change-Id: I4a7ff0e9c420c5dcfdb2b96884e019a5943828a4
Reviewed-on: https://chromium-review.googlesource.com/1201522
Reviewed-by: Georg Neis <neis@chromium.org>
Commit-Queue: Benedikt Meurer <bmeurer@chromium.org>
Cr-Commit-Position: refs/heads/master@{#55595}
[modify] https://crrev.com/b89811227717381e7d50d03cf19fca00f39cc3aa/src/compiler/operation-typer.cc
[modify] https://crrev.com/b89811227717381e7d50d03cf19fca00f39cc3aa/src/compiler/operation-typer.h
[modify] https://crrev.com/b89811227717381e7d50d03cf19fca00f39cc3aa/src/compiler/types.h
[add] https://crrev.com/b89811227717381e7d50d03cf19fca00f39cc3aa/test/mjsunit/regress/regress-crbug-879898.js

Project Member

Comment 24 by bugdroid1@chromium.org, Sep 3

The following revision refers to this bug:
  https://chromium.googlesource.com/v8/v8.git/+/b89811227717381e7d50d03cf19fca00f39cc3aa

commit b89811227717381e7d50d03cf19fca00f39cc3aa
Author: Benedikt Meurer <bmeurer@chromium.org>
Date: Mon Sep 03 19:14:09 2018

[turbofan] Improve typing of ToNumeric and ToNumber.

The previous typing rules for ToNumeric and ToNumber didn't match on the
NonBigIntPrimitive input set, which causes trouble when we morph ToNumeric
nodes into ToNumber nodes, and generally lead to worse typings in the
graph, and thus worse code generation. This change improves the existing
typing rules and turns ToNumber into a chokepoint again.

Bug:  chromium:879898 , v8:8015
Change-Id: I4a7ff0e9c420c5dcfdb2b96884e019a5943828a4
Reviewed-on: https://chromium-review.googlesource.com/1201522
Reviewed-by: Georg Neis <neis@chromium.org>
Commit-Queue: Benedikt Meurer <bmeurer@chromium.org>
Cr-Commit-Position: refs/heads/master@{#55595}
[modify] https://crrev.com/b89811227717381e7d50d03cf19fca00f39cc3aa/src/compiler/operation-typer.cc
[modify] https://crrev.com/b89811227717381e7d50d03cf19fca00f39cc3aa/src/compiler/operation-typer.h
[modify] https://crrev.com/b89811227717381e7d50d03cf19fca00f39cc3aa/src/compiler/types.h
[add] https://crrev.com/b89811227717381e7d50d03cf19fca00f39cc3aa/test/mjsunit/regress/regress-crbug-879898.js

Project Member

Comment 25 by bugdroid1@chromium.org, Sep 3

The following revision refers to this bug:
  https://chromium.googlesource.com/v8/v8.git/+/b89811227717381e7d50d03cf19fca00f39cc3aa

commit b89811227717381e7d50d03cf19fca00f39cc3aa
Author: Benedikt Meurer <bmeurer@chromium.org>
Date: Mon Sep 03 19:14:09 2018

[turbofan] Improve typing of ToNumeric and ToNumber.

The previous typing rules for ToNumeric and ToNumber didn't match on the
NonBigIntPrimitive input set, which causes trouble when we morph ToNumeric
nodes into ToNumber nodes, and generally lead to worse typings in the
graph, and thus worse code generation. This change improves the existing
typing rules and turns ToNumber into a chokepoint again.

Bug:  chromium:879898 , v8:8015
Change-Id: I4a7ff0e9c420c5dcfdb2b96884e019a5943828a4
Reviewed-on: https://chromium-review.googlesource.com/1201522
Reviewed-by: Georg Neis <neis@chromium.org>
Commit-Queue: Benedikt Meurer <bmeurer@chromium.org>
Cr-Commit-Position: refs/heads/master@{#55595}
[modify] https://crrev.com/b89811227717381e7d50d03cf19fca00f39cc3aa/src/compiler/operation-typer.cc
[modify] https://crrev.com/b89811227717381e7d50d03cf19fca00f39cc3aa/src/compiler/operation-typer.h
[modify] https://crrev.com/b89811227717381e7d50d03cf19fca00f39cc3aa/src/compiler/types.h
[add] https://crrev.com/b89811227717381e7d50d03cf19fca00f39cc3aa/test/mjsunit/regress/regress-crbug-879898.js

Project Member

Comment 26 by bugdroid1@chromium.org, Sep 3

The following revision refers to this bug:
  https://chromium.googlesource.com/v8/v8.git/+/b89811227717381e7d50d03cf19fca00f39cc3aa

commit b89811227717381e7d50d03cf19fca00f39cc3aa
Author: Benedikt Meurer <bmeurer@chromium.org>
Date: Mon Sep 03 19:14:09 2018

[turbofan] Improve typing of ToNumeric and ToNumber.

The previous typing rules for ToNumeric and ToNumber didn't match on the
NonBigIntPrimitive input set, which causes trouble when we morph ToNumeric
nodes into ToNumber nodes, and generally lead to worse typings in the
graph, and thus worse code generation. This change improves the existing
typing rules and turns ToNumber into a chokepoint again.

Bug:  chromium:879898 , v8:8015
Change-Id: I4a7ff0e9c420c5dcfdb2b96884e019a5943828a4
Reviewed-on: https://chromium-review.googlesource.com/1201522
Reviewed-by: Georg Neis <neis@chromium.org>
Commit-Queue: Benedikt Meurer <bmeurer@chromium.org>
Cr-Commit-Position: refs/heads/master@{#55595}
[modify] https://crrev.com/b89811227717381e7d50d03cf19fca00f39cc3aa/src/compiler/operation-typer.cc
[modify] https://crrev.com/b89811227717381e7d50d03cf19fca00f39cc3aa/src/compiler/operation-typer.h
[modify] https://crrev.com/b89811227717381e7d50d03cf19fca00f39cc3aa/src/compiler/types.h
[add] https://crrev.com/b89811227717381e7d50d03cf19fca00f39cc3aa/test/mjsunit/regress/regress-crbug-879898.js

Project Member

Comment 27 by bugdroid1@chromium.org, Sep 3

The following revision refers to this bug:
  https://chromium.googlesource.com/v8/v8.git/+/b89811227717381e7d50d03cf19fca00f39cc3aa

commit b89811227717381e7d50d03cf19fca00f39cc3aa
Author: Benedikt Meurer <bmeurer@chromium.org>
Date: Mon Sep 03 19:14:09 2018

[turbofan] Improve typing of ToNumeric and ToNumber.

The previous typing rules for ToNumeric and ToNumber didn't match on the
NonBigIntPrimitive input set, which causes trouble when we morph ToNumeric
nodes into ToNumber nodes, and generally lead to worse typings in the
graph, and thus worse code generation. This change improves the existing
typing rules and turns ToNumber into a chokepoint again.

Bug:  chromium:879898 , v8:8015
Change-Id: I4a7ff0e9c420c5dcfdb2b96884e019a5943828a4
Reviewed-on: https://chromium-review.googlesource.com/1201522
Reviewed-by: Georg Neis <neis@chromium.org>
Commit-Queue: Benedikt Meurer <bmeurer@chromium.org>
Cr-Commit-Position: refs/heads/master@{#55595}
[modify] https://crrev.com/b89811227717381e7d50d03cf19fca00f39cc3aa/src/compiler/operation-typer.cc
[modify] https://crrev.com/b89811227717381e7d50d03cf19fca00f39cc3aa/src/compiler/operation-typer.h
[modify] https://crrev.com/b89811227717381e7d50d03cf19fca00f39cc3aa/src/compiler/types.h
[add] https://crrev.com/b89811227717381e7d50d03cf19fca00f39cc3aa/test/mjsunit/regress/regress-crbug-879898.js

Project Member

Comment 28 by bugdroid1@chromium.org, Sep 3

The following revision refers to this bug:
  https://chromium.googlesource.com/v8/v8.git/+/b89811227717381e7d50d03cf19fca00f39cc3aa

commit b89811227717381e7d50d03cf19fca00f39cc3aa
Author: Benedikt Meurer <bmeurer@chromium.org>
Date: Mon Sep 03 19:14:09 2018

[turbofan] Improve typing of ToNumeric and ToNumber.

The previous typing rules for ToNumeric and ToNumber didn't match on the
NonBigIntPrimitive input set, which causes trouble when we morph ToNumeric
nodes into ToNumber nodes, and generally lead to worse typings in the
graph, and thus worse code generation. This change improves the existing
typing rules and turns ToNumber into a chokepoint again.

Bug:  chromium:879898 , v8:8015
Change-Id: I4a7ff0e9c420c5dcfdb2b96884e019a5943828a4
Reviewed-on: https://chromium-review.googlesource.com/1201522
Reviewed-by: Georg Neis <neis@chromium.org>
Commit-Queue: Benedikt Meurer <bmeurer@chromium.org>
Cr-Commit-Position: refs/heads/master@{#55595}
[modify] https://crrev.com/b89811227717381e7d50d03cf19fca00f39cc3aa/src/compiler/operation-typer.cc
[modify] https://crrev.com/b89811227717381e7d50d03cf19fca00f39cc3aa/src/compiler/operation-typer.h
[modify] https://crrev.com/b89811227717381e7d50d03cf19fca00f39cc3aa/src/compiler/types.h
[add] https://crrev.com/b89811227717381e7d50d03cf19fca00f39cc3aa/test/mjsunit/regress/regress-crbug-879898.js

Project Member

Comment 29 by bugdroid1@chromium.org, Sep 3

The following revision refers to this bug:
  https://chromium.googlesource.com/v8/v8.git/+/b89811227717381e7d50d03cf19fca00f39cc3aa

commit b89811227717381e7d50d03cf19fca00f39cc3aa
Author: Benedikt Meurer <bmeurer@chromium.org>
Date: Mon Sep 03 19:14:09 2018

[turbofan] Improve typing of ToNumeric and ToNumber.

The previous typing rules for ToNumeric and ToNumber didn't match on the
NonBigIntPrimitive input set, which causes trouble when we morph ToNumeric
nodes into ToNumber nodes, and generally lead to worse typings in the
graph, and thus worse code generation. This change improves the existing
typing rules and turns ToNumber into a chokepoint again.

Bug:  chromium:879898 , v8:8015
Change-Id: I4a7ff0e9c420c5dcfdb2b96884e019a5943828a4
Reviewed-on: https://chromium-review.googlesource.com/1201522
Reviewed-by: Georg Neis <neis@chromium.org>
Commit-Queue: Benedikt Meurer <bmeurer@chromium.org>
Cr-Commit-Position: refs/heads/master@{#55595}
[modify] https://crrev.com/b89811227717381e7d50d03cf19fca00f39cc3aa/src/compiler/operation-typer.cc
[modify] https://crrev.com/b89811227717381e7d50d03cf19fca00f39cc3aa/src/compiler/operation-typer.h
[modify] https://crrev.com/b89811227717381e7d50d03cf19fca00f39cc3aa/src/compiler/types.h
[add] https://crrev.com/b89811227717381e7d50d03cf19fca00f39cc3aa/test/mjsunit/regress/regress-crbug-879898.js

Project Member

Comment 30 by bugdroid1@chromium.org, Sep 3

The following revision refers to this bug:
  https://chromium.googlesource.com/v8/v8.git/+/b89811227717381e7d50d03cf19fca00f39cc3aa

commit b89811227717381e7d50d03cf19fca00f39cc3aa
Author: Benedikt Meurer <bmeurer@chromium.org>
Date: Mon Sep 03 19:14:09 2018

[turbofan] Improve typing of ToNumeric and ToNumber.

The previous typing rules for ToNumeric and ToNumber didn't match on the
NonBigIntPrimitive input set, which causes trouble when we morph ToNumeric
nodes into ToNumber nodes, and generally lead to worse typings in the
graph, and thus worse code generation. This change improves the existing
typing rules and turns ToNumber into a chokepoint again.

Bug:  chromium:879898 , v8:8015
Change-Id: I4a7ff0e9c420c5dcfdb2b96884e019a5943828a4
Reviewed-on: https://chromium-review.googlesource.com/1201522
Reviewed-by: Georg Neis <neis@chromium.org>
Commit-Queue: Benedikt Meurer <bmeurer@chromium.org>
Cr-Commit-Position: refs/heads/master@{#55595}
[modify] https://crrev.com/b89811227717381e7d50d03cf19fca00f39cc3aa/src/compiler/operation-typer.cc
[modify] https://crrev.com/b89811227717381e7d50d03cf19fca00f39cc3aa/src/compiler/operation-typer.h
[modify] https://crrev.com/b89811227717381e7d50d03cf19fca00f39cc3aa/src/compiler/types.h
[add] https://crrev.com/b89811227717381e7d50d03cf19fca00f39cc3aa/test/mjsunit/regress/regress-crbug-879898.js

Project Member

Comment 31 by bugdroid1@chromium.org, Sep 3

The following revision refers to this bug:
  https://chromium.googlesource.com/v8/v8.git/+/b89811227717381e7d50d03cf19fca00f39cc3aa

commit b89811227717381e7d50d03cf19fca00f39cc3aa
Author: Benedikt Meurer <bmeurer@chromium.org>
Date: Mon Sep 03 19:14:09 2018

[turbofan] Improve typing of ToNumeric and ToNumber.

The previous typing rules for ToNumeric and ToNumber didn't match on the
NonBigIntPrimitive input set, which causes trouble when we morph ToNumeric
nodes into ToNumber nodes, and generally lead to worse typings in the
graph, and thus worse code generation. This change improves the existing
typing rules and turns ToNumber into a chokepoint again.

Bug:  chromium:879898 , v8:8015
Change-Id: I4a7ff0e9c420c5dcfdb2b96884e019a5943828a4
Reviewed-on: https://chromium-review.googlesource.com/1201522
Reviewed-by: Georg Neis <neis@chromium.org>
Commit-Queue: Benedikt Meurer <bmeurer@chromium.org>
Cr-Commit-Position: refs/heads/master@{#55595}
[modify] https://crrev.com/b89811227717381e7d50d03cf19fca00f39cc3aa/src/compiler/operation-typer.cc
[modify] https://crrev.com/b89811227717381e7d50d03cf19fca00f39cc3aa/src/compiler/operation-typer.h
[modify] https://crrev.com/b89811227717381e7d50d03cf19fca00f39cc3aa/src/compiler/types.h
[add] https://crrev.com/b89811227717381e7d50d03cf19fca00f39cc3aa/test/mjsunit/regress/regress-crbug-879898.js

Project Member

Comment 32 by bugdroid1@chromium.org, Sep 3

The following revision refers to this bug:
  https://chromium.googlesource.com/v8/v8.git/+/b89811227717381e7d50d03cf19fca00f39cc3aa

commit b89811227717381e7d50d03cf19fca00f39cc3aa
Author: Benedikt Meurer <bmeurer@chromium.org>
Date: Mon Sep 03 19:14:09 2018

[turbofan] Improve typing of ToNumeric and ToNumber.

The previous typing rules for ToNumeric and ToNumber didn't match on the
NonBigIntPrimitive input set, which causes trouble when we morph ToNumeric
nodes into ToNumber nodes, and generally lead to worse typings in the
graph, and thus worse code generation. This change improves the existing
typing rules and turns ToNumber into a chokepoint again.

Bug:  chromium:879898 , v8:8015
Change-Id: I4a7ff0e9c420c5dcfdb2b96884e019a5943828a4
Reviewed-on: https://chromium-review.googlesource.com/1201522
Reviewed-by: Georg Neis <neis@chromium.org>
Commit-Queue: Benedikt Meurer <bmeurer@chromium.org>
Cr-Commit-Position: refs/heads/master@{#55595}
[modify] https://crrev.com/b89811227717381e7d50d03cf19fca00f39cc3aa/src/compiler/operation-typer.cc
[modify] https://crrev.com/b89811227717381e7d50d03cf19fca00f39cc3aa/src/compiler/operation-typer.h
[modify] https://crrev.com/b89811227717381e7d50d03cf19fca00f39cc3aa/src/compiler/types.h
[add] https://crrev.com/b89811227717381e7d50d03cf19fca00f39cc3aa/test/mjsunit/regress/regress-crbug-879898.js

Project Member

Comment 33 by bugdroid1@chromium.org, Sep 3

The following revision refers to this bug:
  https://chromium.googlesource.com/v8/v8.git/+/b89811227717381e7d50d03cf19fca00f39cc3aa

commit b89811227717381e7d50d03cf19fca00f39cc3aa
Author: Benedikt Meurer <bmeurer@chromium.org>
Date: Mon Sep 03 19:14:09 2018

[turbofan] Improve typing of ToNumeric and ToNumber.

The previous typing rules for ToNumeric and ToNumber didn't match on the
NonBigIntPrimitive input set, which causes trouble when we morph ToNumeric
nodes into ToNumber nodes, and generally lead to worse typings in the
graph, and thus worse code generation. This change improves the existing
typing rules and turns ToNumber into a chokepoint again.

Bug:  chromium:879898 , v8:8015
Change-Id: I4a7ff0e9c420c5dcfdb2b96884e019a5943828a4
Reviewed-on: https://chromium-review.googlesource.com/1201522
Reviewed-by: Georg Neis <neis@chromium.org>
Commit-Queue: Benedikt Meurer <bmeurer@chromium.org>
Cr-Commit-Position: refs/heads/master@{#55595}
[modify] https://crrev.com/b89811227717381e7d50d03cf19fca00f39cc3aa/src/compiler/operation-typer.cc
[modify] https://crrev.com/b89811227717381e7d50d03cf19fca00f39cc3aa/src/compiler/operation-typer.h
[modify] https://crrev.com/b89811227717381e7d50d03cf19fca00f39cc3aa/src/compiler/types.h
[add] https://crrev.com/b89811227717381e7d50d03cf19fca00f39cc3aa/test/mjsunit/regress/regress-crbug-879898.js

Project Member

Comment 34 by bugdroid1@chromium.org, Sep 3

The following revision refers to this bug:
  https://chromium.googlesource.com/v8/v8.git/+/b89811227717381e7d50d03cf19fca00f39cc3aa

commit b89811227717381e7d50d03cf19fca00f39cc3aa
Author: Benedikt Meurer <bmeurer@chromium.org>
Date: Mon Sep 03 19:14:09 2018

[turbofan] Improve typing of ToNumeric and ToNumber.

The previous typing rules for ToNumeric and ToNumber didn't match on the
NonBigIntPrimitive input set, which causes trouble when we morph ToNumeric
nodes into ToNumber nodes, and generally lead to worse typings in the
graph, and thus worse code generation. This change improves the existing
typing rules and turns ToNumber into a chokepoint again.

Bug:  chromium:879898 , v8:8015
Change-Id: I4a7ff0e9c420c5dcfdb2b96884e019a5943828a4
Reviewed-on: https://chromium-review.googlesource.com/1201522
Reviewed-by: Georg Neis <neis@chromium.org>
Commit-Queue: Benedikt Meurer <bmeurer@chromium.org>
Cr-Commit-Position: refs/heads/master@{#55595}
[modify] https://crrev.com/b89811227717381e7d50d03cf19fca00f39cc3aa/src/compiler/operation-typer.cc
[modify] https://crrev.com/b89811227717381e7d50d03cf19fca00f39cc3aa/src/compiler/operation-typer.h
[modify] https://crrev.com/b89811227717381e7d50d03cf19fca00f39cc3aa/src/compiler/types.h
[add] https://crrev.com/b89811227717381e7d50d03cf19fca00f39cc3aa/test/mjsunit/regress/regress-crbug-879898.js

Project Member

Comment 35 by bugdroid1@chromium.org, Sep 3

The following revision refers to this bug:
  https://chromium.googlesource.com/v8/v8.git/+/b89811227717381e7d50d03cf19fca00f39cc3aa

commit b89811227717381e7d50d03cf19fca00f39cc3aa
Author: Benedikt Meurer <bmeurer@chromium.org>
Date: Mon Sep 03 19:14:09 2018

[turbofan] Improve typing of ToNumeric and ToNumber.

The previous typing rules for ToNumeric and ToNumber didn't match on the
NonBigIntPrimitive input set, which causes trouble when we morph ToNumeric
nodes into ToNumber nodes, and generally lead to worse typings in the
graph, and thus worse code generation. This change improves the existing
typing rules and turns ToNumber into a chokepoint again.

Bug:  chromium:879898 , v8:8015
Change-Id: I4a7ff0e9c420c5dcfdb2b96884e019a5943828a4
Reviewed-on: https://chromium-review.googlesource.com/1201522
Reviewed-by: Georg Neis <neis@chromium.org>
Commit-Queue: Benedikt Meurer <bmeurer@chromium.org>
Cr-Commit-Position: refs/heads/master@{#55595}
[modify] https://crrev.com/b89811227717381e7d50d03cf19fca00f39cc3aa/src/compiler/operation-typer.cc
[modify] https://crrev.com/b89811227717381e7d50d03cf19fca00f39cc3aa/src/compiler/operation-typer.h
[modify] https://crrev.com/b89811227717381e7d50d03cf19fca00f39cc3aa/src/compiler/types.h
[add] https://crrev.com/b89811227717381e7d50d03cf19fca00f39cc3aa/test/mjsunit/regress/regress-crbug-879898.js

Project Member

Comment 36 by bugdroid1@chromium.org, Sep 3

The following revision refers to this bug:
  https://chromium.googlesource.com/v8/v8.git/+/b89811227717381e7d50d03cf19fca00f39cc3aa

commit b89811227717381e7d50d03cf19fca00f39cc3aa
Author: Benedikt Meurer <bmeurer@chromium.org>
Date: Mon Sep 03 19:14:09 2018

[turbofan] Improve typing of ToNumeric and ToNumber.

The previous typing rules for ToNumeric and ToNumber didn't match on the
NonBigIntPrimitive input set, which causes trouble when we morph ToNumeric
nodes into ToNumber nodes, and generally lead to worse typings in the
graph, and thus worse code generation. This change improves the existing
typing rules and turns ToNumber into a chokepoint again.

Bug:  chromium:879898 , v8:8015
Change-Id: I4a7ff0e9c420c5dcfdb2b96884e019a5943828a4
Reviewed-on: https://chromium-review.googlesource.com/1201522
Reviewed-by: Georg Neis <neis@chromium.org>
Commit-Queue: Benedikt Meurer <bmeurer@chromium.org>
Cr-Commit-Position: refs/heads/master@{#55595}
[modify] https://crrev.com/b89811227717381e7d50d03cf19fca00f39cc3aa/src/compiler/operation-typer.cc
[modify] https://crrev.com/b89811227717381e7d50d03cf19fca00f39cc3aa/src/compiler/operation-typer.h
[modify] https://crrev.com/b89811227717381e7d50d03cf19fca00f39cc3aa/src/compiler/types.h
[add] https://crrev.com/b89811227717381e7d50d03cf19fca00f39cc3aa/test/mjsunit/regress/regress-crbug-879898.js

Project Member

Comment 37 by bugdroid1@chromium.org, Sep 4

The following revision refers to this bug:
  https://chromium.googlesource.com/v8/v8.git/+/b89811227717381e7d50d03cf19fca00f39cc3aa

commit b89811227717381e7d50d03cf19fca00f39cc3aa
Author: Benedikt Meurer <bmeurer@chromium.org>
Date: Mon Sep 03 19:14:09 2018

[turbofan] Improve typing of ToNumeric and ToNumber.

The previous typing rules for ToNumeric and ToNumber didn't match on the
NonBigIntPrimitive input set, which causes trouble when we morph ToNumeric
nodes into ToNumber nodes, and generally lead to worse typings in the
graph, and thus worse code generation. This change improves the existing
typing rules and turns ToNumber into a chokepoint again.

Bug:  chromium:879898 , v8:8015
Change-Id: I4a7ff0e9c420c5dcfdb2b96884e019a5943828a4
Reviewed-on: https://chromium-review.googlesource.com/1201522
Reviewed-by: Georg Neis <neis@chromium.org>
Commit-Queue: Benedikt Meurer <bmeurer@chromium.org>
Cr-Commit-Position: refs/heads/master@{#55595}
[modify] https://crrev.com/b89811227717381e7d50d03cf19fca00f39cc3aa/src/compiler/operation-typer.cc
[modify] https://crrev.com/b89811227717381e7d50d03cf19fca00f39cc3aa/src/compiler/operation-typer.h
[modify] https://crrev.com/b89811227717381e7d50d03cf19fca00f39cc3aa/src/compiler/types.h
[add] https://crrev.com/b89811227717381e7d50d03cf19fca00f39cc3aa/test/mjsunit/regress/regress-crbug-879898.js

Project Member

Comment 38 by bugdroid1@chromium.org, Sep 4

The following revision refers to this bug:
  https://chromium.googlesource.com/v8/v8.git/+/b89811227717381e7d50d03cf19fca00f39cc3aa

commit b89811227717381e7d50d03cf19fca00f39cc3aa
Author: Benedikt Meurer <bmeurer@chromium.org>
Date: Mon Sep 03 19:14:09 2018

[turbofan] Improve typing of ToNumeric and ToNumber.

The previous typing rules for ToNumeric and ToNumber didn't match on the
NonBigIntPrimitive input set, which causes trouble when we morph ToNumeric
nodes into ToNumber nodes, and generally lead to worse typings in the
graph, and thus worse code generation. This change improves the existing
typing rules and turns ToNumber into a chokepoint again.

Bug:  chromium:879898 , v8:8015
Change-Id: I4a7ff0e9c420c5dcfdb2b96884e019a5943828a4
Reviewed-on: https://chromium-review.googlesource.com/1201522
Reviewed-by: Georg Neis <neis@chromium.org>
Commit-Queue: Benedikt Meurer <bmeurer@chromium.org>
Cr-Commit-Position: refs/heads/master@{#55595}
[modify] https://crrev.com/b89811227717381e7d50d03cf19fca00f39cc3aa/src/compiler/operation-typer.cc
[modify] https://crrev.com/b89811227717381e7d50d03cf19fca00f39cc3aa/src/compiler/operation-typer.h
[modify] https://crrev.com/b89811227717381e7d50d03cf19fca00f39cc3aa/src/compiler/types.h
[add] https://crrev.com/b89811227717381e7d50d03cf19fca00f39cc3aa/test/mjsunit/regress/regress-crbug-879898.js

Project Member

Comment 39 by bugdroid1@chromium.org, Sep 4

The following revision refers to this bug:
  https://chromium.googlesource.com/v8/v8.git/+/b89811227717381e7d50d03cf19fca00f39cc3aa

commit b89811227717381e7d50d03cf19fca00f39cc3aa
Author: Benedikt Meurer <bmeurer@chromium.org>
Date: Mon Sep 03 19:14:09 2018

[turbofan] Improve typing of ToNumeric and ToNumber.

The previous typing rules for ToNumeric and ToNumber didn't match on the
NonBigIntPrimitive input set, which causes trouble when we morph ToNumeric
nodes into ToNumber nodes, and generally lead to worse typings in the
graph, and thus worse code generation. This change improves the existing
typing rules and turns ToNumber into a chokepoint again.

Bug:  chromium:879898 , v8:8015
Change-Id: I4a7ff0e9c420c5dcfdb2b96884e019a5943828a4
Reviewed-on: https://chromium-review.googlesource.com/1201522
Reviewed-by: Georg Neis <neis@chromium.org>
Commit-Queue: Benedikt Meurer <bmeurer@chromium.org>
Cr-Commit-Position: refs/heads/master@{#55595}
[modify] https://crrev.com/b89811227717381e7d50d03cf19fca00f39cc3aa/src/compiler/operation-typer.cc
[modify] https://crrev.com/b89811227717381e7d50d03cf19fca00f39cc3aa/src/compiler/operation-typer.h
[modify] https://crrev.com/b89811227717381e7d50d03cf19fca00f39cc3aa/src/compiler/types.h
[add] https://crrev.com/b89811227717381e7d50d03cf19fca00f39cc3aa/test/mjsunit/regress/regress-crbug-879898.js

Project Member

Comment 40 by bugdroid1@chromium.org, Sep 4

The following revision refers to this bug:
  https://chromium.googlesource.com/v8/v8.git/+/b89811227717381e7d50d03cf19fca00f39cc3aa

commit b89811227717381e7d50d03cf19fca00f39cc3aa
Author: Benedikt Meurer <bmeurer@chromium.org>
Date: Mon Sep 03 19:14:09 2018

[turbofan] Improve typing of ToNumeric and ToNumber.

The previous typing rules for ToNumeric and ToNumber didn't match on the
NonBigIntPrimitive input set, which causes trouble when we morph ToNumeric
nodes into ToNumber nodes, and generally lead to worse typings in the
graph, and thus worse code generation. This change improves the existing
typing rules and turns ToNumber into a chokepoint again.

Bug:  chromium:879898 , v8:8015
Change-Id: I4a7ff0e9c420c5dcfdb2b96884e019a5943828a4
Reviewed-on: https://chromium-review.googlesource.com/1201522
Reviewed-by: Georg Neis <neis@chromium.org>
Commit-Queue: Benedikt Meurer <bmeurer@chromium.org>
Cr-Commit-Position: refs/heads/master@{#55595}
[modify] https://crrev.com/b89811227717381e7d50d03cf19fca00f39cc3aa/src/compiler/operation-typer.cc
[modify] https://crrev.com/b89811227717381e7d50d03cf19fca00f39cc3aa/src/compiler/operation-typer.h
[modify] https://crrev.com/b89811227717381e7d50d03cf19fca00f39cc3aa/src/compiler/types.h
[add] https://crrev.com/b89811227717381e7d50d03cf19fca00f39cc3aa/test/mjsunit/regress/regress-crbug-879898.js

Project Member

Comment 41 by bugdroid1@chromium.org, Sep 4

The following revision refers to this bug:
  https://chromium.googlesource.com/v8/v8.git/+/b89811227717381e7d50d03cf19fca00f39cc3aa

commit b89811227717381e7d50d03cf19fca00f39cc3aa
Author: Benedikt Meurer <bmeurer@chromium.org>
Date: Mon Sep 03 19:14:09 2018

[turbofan] Improve typing of ToNumeric and ToNumber.

The previous typing rules for ToNumeric and ToNumber didn't match on the
NonBigIntPrimitive input set, which causes trouble when we morph ToNumeric
nodes into ToNumber nodes, and generally lead to worse typings in the
graph, and thus worse code generation. This change improves the existing
typing rules and turns ToNumber into a chokepoint again.

Bug:  chromium:879898 , v8:8015
Change-Id: I4a7ff0e9c420c5dcfdb2b96884e019a5943828a4
Reviewed-on: https://chromium-review.googlesource.com/1201522
Reviewed-by: Georg Neis <neis@chromium.org>
Commit-Queue: Benedikt Meurer <bmeurer@chromium.org>
Cr-Commit-Position: refs/heads/master@{#55595}
[modify] https://crrev.com/b89811227717381e7d50d03cf19fca00f39cc3aa/src/compiler/operation-typer.cc
[modify] https://crrev.com/b89811227717381e7d50d03cf19fca00f39cc3aa/src/compiler/operation-typer.h
[modify] https://crrev.com/b89811227717381e7d50d03cf19fca00f39cc3aa/src/compiler/types.h
[add] https://crrev.com/b89811227717381e7d50d03cf19fca00f39cc3aa/test/mjsunit/regress/regress-crbug-879898.js

Project Member

Comment 42 by bugdroid1@chromium.org, Sep 4

The following revision refers to this bug:
  https://chromium.googlesource.com/v8/v8.git/+/b89811227717381e7d50d03cf19fca00f39cc3aa

commit b89811227717381e7d50d03cf19fca00f39cc3aa
Author: Benedikt Meurer <bmeurer@chromium.org>
Date: Mon Sep 03 19:14:09 2018

[turbofan] Improve typing of ToNumeric and ToNumber.

The previous typing rules for ToNumeric and ToNumber didn't match on the
NonBigIntPrimitive input set, which causes trouble when we morph ToNumeric
nodes into ToNumber nodes, and generally lead to worse typings in the
graph, and thus worse code generation. This change improves the existing
typing rules and turns ToNumber into a chokepoint again.

Bug:  chromium:879898 , v8:8015
Change-Id: I4a7ff0e9c420c5dcfdb2b96884e019a5943828a4
Reviewed-on: https://chromium-review.googlesource.com/1201522
Reviewed-by: Georg Neis <neis@chromium.org>
Commit-Queue: Benedikt Meurer <bmeurer@chromium.org>
Cr-Commit-Position: refs/heads/master@{#55595}
[modify] https://crrev.com/b89811227717381e7d50d03cf19fca00f39cc3aa/src/compiler/operation-typer.cc
[modify] https://crrev.com/b89811227717381e7d50d03cf19fca00f39cc3aa/src/compiler/operation-typer.h
[modify] https://crrev.com/b89811227717381e7d50d03cf19fca00f39cc3aa/src/compiler/types.h
[add] https://crrev.com/b89811227717381e7d50d03cf19fca00f39cc3aa/test/mjsunit/regress/regress-crbug-879898.js

Project Member

Comment 43 by bugdroid1@chromium.org, Sep 4

The following revision refers to this bug:
  https://chromium.googlesource.com/v8/v8.git/+/b89811227717381e7d50d03cf19fca00f39cc3aa

commit b89811227717381e7d50d03cf19fca00f39cc3aa
Author: Benedikt Meurer <bmeurer@chromium.org>
Date: Mon Sep 03 19:14:09 2018

[turbofan] Improve typing of ToNumeric and ToNumber.

The previous typing rules for ToNumeric and ToNumber didn't match on the
NonBigIntPrimitive input set, which causes trouble when we morph ToNumeric
nodes into ToNumber nodes, and generally lead to worse typings in the
graph, and thus worse code generation. This change improves the existing
typing rules and turns ToNumber into a chokepoint again.

Bug:  chromium:879898 , v8:8015
Change-Id: I4a7ff0e9c420c5dcfdb2b96884e019a5943828a4
Reviewed-on: https://chromium-review.googlesource.com/1201522
Reviewed-by: Georg Neis <neis@chromium.org>
Commit-Queue: Benedikt Meurer <bmeurer@chromium.org>
Cr-Commit-Position: refs/heads/master@{#55595}
[modify] https://crrev.com/b89811227717381e7d50d03cf19fca00f39cc3aa/src/compiler/operation-typer.cc
[modify] https://crrev.com/b89811227717381e7d50d03cf19fca00f39cc3aa/src/compiler/operation-typer.h
[modify] https://crrev.com/b89811227717381e7d50d03cf19fca00f39cc3aa/src/compiler/types.h
[add] https://crrev.com/b89811227717381e7d50d03cf19fca00f39cc3aa/test/mjsunit/regress/regress-crbug-879898.js

Project Member

Comment 44 by bugdroid1@chromium.org, Sep 4

The following revision refers to this bug:
  https://chromium.googlesource.com/v8/v8.git/+/b89811227717381e7d50d03cf19fca00f39cc3aa

commit b89811227717381e7d50d03cf19fca00f39cc3aa
Author: Benedikt Meurer <bmeurer@chromium.org>
Date: Mon Sep 03 19:14:09 2018

[turbofan] Improve typing of ToNumeric and ToNumber.

The previous typing rules for ToNumeric and ToNumber didn't match on the
NonBigIntPrimitive input set, which causes trouble when we morph ToNumeric
nodes into ToNumber nodes, and generally lead to worse typings in the
graph, and thus worse code generation. This change improves the existing
typing rules and turns ToNumber into a chokepoint again.

Bug:  chromium:879898 , v8:8015
Change-Id: I4a7ff0e9c420c5dcfdb2b96884e019a5943828a4
Reviewed-on: https://chromium-review.googlesource.com/1201522
Reviewed-by: Georg Neis <neis@chromium.org>
Commit-Queue: Benedikt Meurer <bmeurer@chromium.org>
Cr-Commit-Position: refs/heads/master@{#55595}
[modify] https://crrev.com/b89811227717381e7d50d03cf19fca00f39cc3aa/src/compiler/operation-typer.cc
[modify] https://crrev.com/b89811227717381e7d50d03cf19fca00f39cc3aa/src/compiler/operation-typer.h
[modify] https://crrev.com/b89811227717381e7d50d03cf19fca00f39cc3aa/src/compiler/types.h
[add] https://crrev.com/b89811227717381e7d50d03cf19fca00f39cc3aa/test/mjsunit/regress/regress-crbug-879898.js

Project Member

Comment 45 by bugdroid1@chromium.org, Sep 4

The following revision refers to this bug:
  https://chromium.googlesource.com/v8/v8.git/+/b89811227717381e7d50d03cf19fca00f39cc3aa

commit b89811227717381e7d50d03cf19fca00f39cc3aa
Author: Benedikt Meurer <bmeurer@chromium.org>
Date: Mon Sep 03 19:14:09 2018

[turbofan] Improve typing of ToNumeric and ToNumber.

The previous typing rules for ToNumeric and ToNumber didn't match on the
NonBigIntPrimitive input set, which causes trouble when we morph ToNumeric
nodes into ToNumber nodes, and generally lead to worse typings in the
graph, and thus worse code generation. This change improves the existing
typing rules and turns ToNumber into a chokepoint again.

Bug:  chromium:879898 , v8:8015
Change-Id: I4a7ff0e9c420c5dcfdb2b96884e019a5943828a4
Reviewed-on: https://chromium-review.googlesource.com/1201522
Reviewed-by: Georg Neis <neis@chromium.org>
Commit-Queue: Benedikt Meurer <bmeurer@chromium.org>
Cr-Commit-Position: refs/heads/master@{#55595}
[modify] https://crrev.com/b89811227717381e7d50d03cf19fca00f39cc3aa/src/compiler/operation-typer.cc
[modify] https://crrev.com/b89811227717381e7d50d03cf19fca00f39cc3aa/src/compiler/operation-typer.h
[modify] https://crrev.com/b89811227717381e7d50d03cf19fca00f39cc3aa/src/compiler/types.h
[add] https://crrev.com/b89811227717381e7d50d03cf19fca00f39cc3aa/test/mjsunit/regress/regress-crbug-879898.js

Project Member

Comment 46 by bugdroid1@chromium.org, Sep 4

The following revision refers to this bug:
  https://chromium.googlesource.com/v8/v8.git/+/b89811227717381e7d50d03cf19fca00f39cc3aa

commit b89811227717381e7d50d03cf19fca00f39cc3aa
Author: Benedikt Meurer <bmeurer@chromium.org>
Date: Mon Sep 03 19:14:09 2018

[turbofan] Improve typing of ToNumeric and ToNumber.

The previous typing rules for ToNumeric and ToNumber didn't match on the
NonBigIntPrimitive input set, which causes trouble when we morph ToNumeric
nodes into ToNumber nodes, and generally lead to worse typings in the
graph, and thus worse code generation. This change improves the existing
typing rules and turns ToNumber into a chokepoint again.

Bug:  chromium:879898 , v8:8015
Change-Id: I4a7ff0e9c420c5dcfdb2b96884e019a5943828a4
Reviewed-on: https://chromium-review.googlesource.com/1201522
Reviewed-by: Georg Neis <neis@chromium.org>
Commit-Queue: Benedikt Meurer <bmeurer@chromium.org>
Cr-Commit-Position: refs/heads/master@{#55595}
[modify] https://crrev.com/b89811227717381e7d50d03cf19fca00f39cc3aa/src/compiler/operation-typer.cc
[modify] https://crrev.com/b89811227717381e7d50d03cf19fca00f39cc3aa/src/compiler/operation-typer.h
[modify] https://crrev.com/b89811227717381e7d50d03cf19fca00f39cc3aa/src/compiler/types.h
[add] https://crrev.com/b89811227717381e7d50d03cf19fca00f39cc3aa/test/mjsunit/regress/regress-crbug-879898.js

Project Member

Comment 47 by bugdroid1@chromium.org, Sep 4

The following revision refers to this bug:
  https://chromium.googlesource.com/v8/v8.git/+/b89811227717381e7d50d03cf19fca00f39cc3aa

commit b89811227717381e7d50d03cf19fca00f39cc3aa
Author: Benedikt Meurer <bmeurer@chromium.org>
Date: Mon Sep 03 19:14:09 2018

[turbofan] Improve typing of ToNumeric and ToNumber.

The previous typing rules for ToNumeric and ToNumber didn't match on the
NonBigIntPrimitive input set, which causes trouble when we morph ToNumeric
nodes into ToNumber nodes, and generally lead to worse typings in the
graph, and thus worse code generation. This change improves the existing
typing rules and turns ToNumber into a chokepoint again.

Bug:  chromium:879898 , v8:8015
Change-Id: I4a7ff0e9c420c5dcfdb2b96884e019a5943828a4
Reviewed-on: https://chromium-review.googlesource.com/1201522
Reviewed-by: Georg Neis <neis@chromium.org>
Commit-Queue: Benedikt Meurer <bmeurer@chromium.org>
Cr-Commit-Position: refs/heads/master@{#55595}
[modify] https://crrev.com/b89811227717381e7d50d03cf19fca00f39cc3aa/src/compiler/operation-typer.cc
[modify] https://crrev.com/b89811227717381e7d50d03cf19fca00f39cc3aa/src/compiler/operation-typer.h
[modify] https://crrev.com/b89811227717381e7d50d03cf19fca00f39cc3aa/src/compiler/types.h
[add] https://crrev.com/b89811227717381e7d50d03cf19fca00f39cc3aa/test/mjsunit/regress/regress-crbug-879898.js

Project Member

Comment 48 by bugdroid1@chromium.org, Sep 4

The following revision refers to this bug:
  https://chromium.googlesource.com/v8/v8.git/+/b89811227717381e7d50d03cf19fca00f39cc3aa

commit b89811227717381e7d50d03cf19fca00f39cc3aa
Author: Benedikt Meurer <bmeurer@chromium.org>
Date: Mon Sep 03 19:14:09 2018

[turbofan] Improve typing of ToNumeric and ToNumber.

The previous typing rules for ToNumeric and ToNumber didn't match on the
NonBigIntPrimitive input set, which causes trouble when we morph ToNumeric
nodes into ToNumber nodes, and generally lead to worse typings in the
graph, and thus worse code generation. This change improves the existing
typing rules and turns ToNumber into a chokepoint again.

Bug:  chromium:879898 , v8:8015
Change-Id: I4a7ff0e9c420c5dcfdb2b96884e019a5943828a4
Reviewed-on: https://chromium-review.googlesource.com/1201522
Reviewed-by: Georg Neis <neis@chromium.org>
Commit-Queue: Benedikt Meurer <bmeurer@chromium.org>
Cr-Commit-Position: refs/heads/master@{#55595}
[modify] https://crrev.com/b89811227717381e7d50d03cf19fca00f39cc3aa/src/compiler/operation-typer.cc
[modify] https://crrev.com/b89811227717381e7d50d03cf19fca00f39cc3aa/src/compiler/operation-typer.h
[modify] https://crrev.com/b89811227717381e7d50d03cf19fca00f39cc3aa/src/compiler/types.h
[add] https://crrev.com/b89811227717381e7d50d03cf19fca00f39cc3aa/test/mjsunit/regress/regress-crbug-879898.js

Project Member

Comment 49 by bugdroid1@chromium.org, Sep 4

The following revision refers to this bug:
  https://chromium.googlesource.com/v8/v8.git/+/b89811227717381e7d50d03cf19fca00f39cc3aa

commit b89811227717381e7d50d03cf19fca00f39cc3aa
Author: Benedikt Meurer <bmeurer@chromium.org>
Date: Mon Sep 03 19:14:09 2018

[turbofan] Improve typing of ToNumeric and ToNumber.

The previous typing rules for ToNumeric and ToNumber didn't match on the
NonBigIntPrimitive input set, which causes trouble when we morph ToNumeric
nodes into ToNumber nodes, and generally lead to worse typings in the
graph, and thus worse code generation. This change improves the existing
typing rules and turns ToNumber into a chokepoint again.

Bug:  chromium:879898 , v8:8015
Change-Id: I4a7ff0e9c420c5dcfdb2b96884e019a5943828a4
Reviewed-on: https://chromium-review.googlesource.com/1201522
Reviewed-by: Georg Neis <neis@chromium.org>
Commit-Queue: Benedikt Meurer <bmeurer@chromium.org>
Cr-Commit-Position: refs/heads/master@{#55595}
[modify] https://crrev.com/b89811227717381e7d50d03cf19fca00f39cc3aa/src/compiler/operation-typer.cc
[modify] https://crrev.com/b89811227717381e7d50d03cf19fca00f39cc3aa/src/compiler/operation-typer.h
[modify] https://crrev.com/b89811227717381e7d50d03cf19fca00f39cc3aa/src/compiler/types.h
[add] https://crrev.com/b89811227717381e7d50d03cf19fca00f39cc3aa/test/mjsunit/regress/regress-crbug-879898.js

Project Member

Comment 50 by bugdroid1@chromium.org, Sep 4

The following revision refers to this bug:
  https://chromium.googlesource.com/v8/v8.git/+/b89811227717381e7d50d03cf19fca00f39cc3aa

commit b89811227717381e7d50d03cf19fca00f39cc3aa
Author: Benedikt Meurer <bmeurer@chromium.org>
Date: Mon Sep 03 19:14:09 2018

[turbofan] Improve typing of ToNumeric and ToNumber.

The previous typing rules for ToNumeric and ToNumber didn't match on the
NonBigIntPrimitive input set, which causes trouble when we morph ToNumeric
nodes into ToNumber nodes, and generally lead to worse typings in the
graph, and thus worse code generation. This change improves the existing
typing rules and turns ToNumber into a chokepoint again.

Bug:  chromium:879898 , v8:8015
Change-Id: I4a7ff0e9c420c5dcfdb2b96884e019a5943828a4
Reviewed-on: https://chromium-review.googlesource.com/1201522
Reviewed-by: Georg Neis <neis@chromium.org>
Commit-Queue: Benedikt Meurer <bmeurer@chromium.org>
Cr-Commit-Position: refs/heads/master@{#55595}
[modify] https://crrev.com/b89811227717381e7d50d03cf19fca00f39cc3aa/src/compiler/operation-typer.cc
[modify] https://crrev.com/b89811227717381e7d50d03cf19fca00f39cc3aa/src/compiler/operation-typer.h
[modify] https://crrev.com/b89811227717381e7d50d03cf19fca00f39cc3aa/src/compiler/types.h
[add] https://crrev.com/b89811227717381e7d50d03cf19fca00f39cc3aa/test/mjsunit/regress/regress-crbug-879898.js

Project Member

Comment 51 by bugdroid1@chromium.org, Sep 4

The following revision refers to this bug:
  https://chromium.googlesource.com/v8/v8.git/+/b89811227717381e7d50d03cf19fca00f39cc3aa

commit b89811227717381e7d50d03cf19fca00f39cc3aa
Author: Benedikt Meurer <bmeurer@chromium.org>
Date: Mon Sep 03 19:14:09 2018

[turbofan] Improve typing of ToNumeric and ToNumber.

The previous typing rules for ToNumeric and ToNumber didn't match on the
NonBigIntPrimitive input set, which causes trouble when we morph ToNumeric
nodes into ToNumber nodes, and generally lead to worse typings in the
graph, and thus worse code generation. This change improves the existing
typing rules and turns ToNumber into a chokepoint again.

Bug:  chromium:879898 , v8:8015
Change-Id: I4a7ff0e9c420c5dcfdb2b96884e019a5943828a4
Reviewed-on: https://chromium-review.googlesource.com/1201522
Reviewed-by: Georg Neis <neis@chromium.org>
Commit-Queue: Benedikt Meurer <bmeurer@chromium.org>
Cr-Commit-Position: refs/heads/master@{#55595}
[modify] https://crrev.com/b89811227717381e7d50d03cf19fca00f39cc3aa/src/compiler/operation-typer.cc
[modify] https://crrev.com/b89811227717381e7d50d03cf19fca00f39cc3aa/src/compiler/operation-typer.h
[modify] https://crrev.com/b89811227717381e7d50d03cf19fca00f39cc3aa/src/compiler/types.h
[add] https://crrev.com/b89811227717381e7d50d03cf19fca00f39cc3aa/test/mjsunit/regress/regress-crbug-879898.js

Project Member

Comment 52 by bugdroid1@chromium.org, Sep 4

The following revision refers to this bug:
  https://chromium.googlesource.com/v8/v8.git/+/b89811227717381e7d50d03cf19fca00f39cc3aa

commit b89811227717381e7d50d03cf19fca00f39cc3aa
Author: Benedikt Meurer <bmeurer@chromium.org>
Date: Mon Sep 03 19:14:09 2018

[turbofan] Improve typing of ToNumeric and ToNumber.

The previous typing rules for ToNumeric and ToNumber didn't match on the
NonBigIntPrimitive input set, which causes trouble when we morph ToNumeric
nodes into ToNumber nodes, and generally lead to worse typings in the
graph, and thus worse code generation. This change improves the existing
typing rules and turns ToNumber into a chokepoint again.

Bug:  chromium:879898 , v8:8015
Change-Id: I4a7ff0e9c420c5dcfdb2b96884e019a5943828a4
Reviewed-on: https://chromium-review.googlesource.com/1201522
Reviewed-by: Georg Neis <neis@chromium.org>
Commit-Queue: Benedikt Meurer <bmeurer@chromium.org>
Cr-Commit-Position: refs/heads/master@{#55595}
[modify] https://crrev.com/b89811227717381e7d50d03cf19fca00f39cc3aa/src/compiler/operation-typer.cc
[modify] https://crrev.com/b89811227717381e7d50d03cf19fca00f39cc3aa/src/compiler/operation-typer.h
[modify] https://crrev.com/b89811227717381e7d50d03cf19fca00f39cc3aa/src/compiler/types.h
[add] https://crrev.com/b89811227717381e7d50d03cf19fca00f39cc3aa/test/mjsunit/regress/regress-crbug-879898.js

Project Member

Comment 53 by bugdroid1@chromium.org, Sep 4

The following revision refers to this bug:
  https://chromium.googlesource.com/v8/v8.git/+/b89811227717381e7d50d03cf19fca00f39cc3aa

commit b89811227717381e7d50d03cf19fca00f39cc3aa
Author: Benedikt Meurer <bmeurer@chromium.org>
Date: Mon Sep 03 19:14:09 2018

[turbofan] Improve typing of ToNumeric and ToNumber.

The previous typing rules for ToNumeric and ToNumber didn't match on the
NonBigIntPrimitive input set, which causes trouble when we morph ToNumeric
nodes into ToNumber nodes, and generally lead to worse typings in the
graph, and thus worse code generation. This change improves the existing
typing rules and turns ToNumber into a chokepoint again.

Bug:  chromium:879898 , v8:8015
Change-Id: I4a7ff0e9c420c5dcfdb2b96884e019a5943828a4
Reviewed-on: https://chromium-review.googlesource.com/1201522
Reviewed-by: Georg Neis <neis@chromium.org>
Commit-Queue: Benedikt Meurer <bmeurer@chromium.org>
Cr-Commit-Position: refs/heads/master@{#55595}
[modify] https://crrev.com/b89811227717381e7d50d03cf19fca00f39cc3aa/src/compiler/operation-typer.cc
[modify] https://crrev.com/b89811227717381e7d50d03cf19fca00f39cc3aa/src/compiler/operation-typer.h
[modify] https://crrev.com/b89811227717381e7d50d03cf19fca00f39cc3aa/src/compiler/types.h
[add] https://crrev.com/b89811227717381e7d50d03cf19fca00f39cc3aa/test/mjsunit/regress/regress-crbug-879898.js

Project Member

Comment 54 by bugdroid1@chromium.org, Sep 4

The following revision refers to this bug:
  https://chromium.googlesource.com/v8/v8.git/+/b89811227717381e7d50d03cf19fca00f39cc3aa

commit b89811227717381e7d50d03cf19fca00f39cc3aa
Author: Benedikt Meurer <bmeurer@chromium.org>
Date: Mon Sep 03 19:14:09 2018

[turbofan] Improve typing of ToNumeric and ToNumber.

The previous typing rules for ToNumeric and ToNumber didn't match on the
NonBigIntPrimitive input set, which causes trouble when we morph ToNumeric
nodes into ToNumber nodes, and generally lead to worse typings in the
graph, and thus worse code generation. This change improves the existing
typing rules and turns ToNumber into a chokepoint again.

Bug:  chromium:879898 , v8:8015
Change-Id: I4a7ff0e9c420c5dcfdb2b96884e019a5943828a4
Reviewed-on: https://chromium-review.googlesource.com/1201522
Reviewed-by: Georg Neis <neis@chromium.org>
Commit-Queue: Benedikt Meurer <bmeurer@chromium.org>
Cr-Commit-Position: refs/heads/master@{#55595}
[modify] https://crrev.com/b89811227717381e7d50d03cf19fca00f39cc3aa/src/compiler/operation-typer.cc
[modify] https://crrev.com/b89811227717381e7d50d03cf19fca00f39cc3aa/src/compiler/operation-typer.h
[modify] https://crrev.com/b89811227717381e7d50d03cf19fca00f39cc3aa/src/compiler/types.h
[add] https://crrev.com/b89811227717381e7d50d03cf19fca00f39cc3aa/test/mjsunit/regress/regress-crbug-879898.js

Project Member

Comment 55 by bugdroid1@chromium.org, Sep 4

The following revision refers to this bug:
  https://chromium.googlesource.com/v8/v8.git/+/b89811227717381e7d50d03cf19fca00f39cc3aa

commit b89811227717381e7d50d03cf19fca00f39cc3aa
Author: Benedikt Meurer <bmeurer@chromium.org>
Date: Mon Sep 03 19:14:09 2018

[turbofan] Improve typing of ToNumeric and ToNumber.

The previous typing rules for ToNumeric and ToNumber didn't match on the
NonBigIntPrimitive input set, which causes trouble when we morph ToNumeric
nodes into ToNumber nodes, and generally lead to worse typings in the
graph, and thus worse code generation. This change improves the existing
typing rules and turns ToNumber into a chokepoint again.

Bug:  chromium:879898 , v8:8015
Change-Id: I4a7ff0e9c420c5dcfdb2b96884e019a5943828a4
Reviewed-on: https://chromium-review.googlesource.com/1201522
Reviewed-by: Georg Neis <neis@chromium.org>
Commit-Queue: Benedikt Meurer <bmeurer@chromium.org>
Cr-Commit-Position: refs/heads/master@{#55595}
[modify] https://crrev.com/b89811227717381e7d50d03cf19fca00f39cc3aa/src/compiler/operation-typer.cc
[modify] https://crrev.com/b89811227717381e7d50d03cf19fca00f39cc3aa/src/compiler/operation-typer.h
[modify] https://crrev.com/b89811227717381e7d50d03cf19fca00f39cc3aa/src/compiler/types.h
[add] https://crrev.com/b89811227717381e7d50d03cf19fca00f39cc3aa/test/mjsunit/regress/regress-crbug-879898.js

Project Member

Comment 56 by bugdroid1@chromium.org, Sep 4

The following revision refers to this bug:
  https://chromium.googlesource.com/v8/v8.git/+/b89811227717381e7d50d03cf19fca00f39cc3aa

commit b89811227717381e7d50d03cf19fca00f39cc3aa
Author: Benedikt Meurer <bmeurer@chromium.org>
Date: Mon Sep 03 19:14:09 2018

[turbofan] Improve typing of ToNumeric and ToNumber.

The previous typing rules for ToNumeric and ToNumber didn't match on the
NonBigIntPrimitive input set, which causes trouble when we morph ToNumeric
nodes into ToNumber nodes, and generally lead to worse typings in the
graph, and thus worse code generation. This change improves the existing
typing rules and turns ToNumber into a chokepoint again.

Bug:  chromium:879898 , v8:8015
Change-Id: I4a7ff0e9c420c5dcfdb2b96884e019a5943828a4
Reviewed-on: https://chromium-review.googlesource.com/1201522
Reviewed-by: Georg Neis <neis@chromium.org>
Commit-Queue: Benedikt Meurer <bmeurer@chromium.org>
Cr-Commit-Position: refs/heads/master@{#55595}
[modify] https://crrev.com/b89811227717381e7d50d03cf19fca00f39cc3aa/src/compiler/operation-typer.cc
[modify] https://crrev.com/b89811227717381e7d50d03cf19fca00f39cc3aa/src/compiler/operation-typer.h
[modify] https://crrev.com/b89811227717381e7d50d03cf19fca00f39cc3aa/src/compiler/types.h
[add] https://crrev.com/b89811227717381e7d50d03cf19fca00f39cc3aa/test/mjsunit/regress/regress-crbug-879898.js

Project Member

Comment 57 by bugdroid1@chromium.org, Sep 4

The following revision refers to this bug:
  https://chromium.googlesource.com/v8/v8.git/+/b89811227717381e7d50d03cf19fca00f39cc3aa

commit b89811227717381e7d50d03cf19fca00f39cc3aa
Author: Benedikt Meurer <bmeurer@chromium.org>
Date: Mon Sep 03 19:14:09 2018

[turbofan] Improve typing of ToNumeric and ToNumber.

The previous typing rules for ToNumeric and ToNumber didn't match on the
NonBigIntPrimitive input set, which causes trouble when we morph ToNumeric
nodes into ToNumber nodes, and generally lead to worse typings in the
graph, and thus worse code generation. This change improves the existing
typing rules and turns ToNumber into a chokepoint again.

Bug:  chromium:879898 , v8:8015
Change-Id: I4a7ff0e9c420c5dcfdb2b96884e019a5943828a4
Reviewed-on: https://chromium-review.googlesource.com/1201522
Reviewed-by: Georg Neis <neis@chromium.org>
Commit-Queue: Benedikt Meurer <bmeurer@chromium.org>
Cr-Commit-Position: refs/heads/master@{#55595}
[modify] https://crrev.com/b89811227717381e7d50d03cf19fca00f39cc3aa/src/compiler/operation-typer.cc
[modify] https://crrev.com/b89811227717381e7d50d03cf19fca00f39cc3aa/src/compiler/operation-typer.h
[modify] https://crrev.com/b89811227717381e7d50d03cf19fca00f39cc3aa/src/compiler/types.h
[add] https://crrev.com/b89811227717381e7d50d03cf19fca00f39cc3aa/test/mjsunit/regress/regress-crbug-879898.js

Project Member

Comment 58 by bugdroid1@chromium.org, Sep 4

The following revision refers to this bug:
  https://chromium.googlesource.com/v8/v8.git/+/b89811227717381e7d50d03cf19fca00f39cc3aa

commit b89811227717381e7d50d03cf19fca00f39cc3aa
Author: Benedikt Meurer <bmeurer@chromium.org>
Date: Mon Sep 03 19:14:09 2018

[turbofan] Improve typing of ToNumeric and ToNumber.

The previous typing rules for ToNumeric and ToNumber didn't match on the
NonBigIntPrimitive input set, which causes trouble when we morph ToNumeric
nodes into ToNumber nodes, and generally lead to worse typings in the
graph, and thus worse code generation. This change improves the existing
typing rules and turns ToNumber into a chokepoint again.

Bug:  chromium:879898 , v8:8015
Change-Id: I4a7ff0e9c420c5dcfdb2b96884e019a5943828a4
Reviewed-on: https://chromium-review.googlesource.com/1201522
Reviewed-by: Georg Neis <neis@chromium.org>
Commit-Queue: Benedikt Meurer <bmeurer@chromium.org>
Cr-Commit-Position: refs/heads/master@{#55595}
[modify] https://crrev.com/b89811227717381e7d50d03cf19fca00f39cc3aa/src/compiler/operation-typer.cc
[modify] https://crrev.com/b89811227717381e7d50d03cf19fca00f39cc3aa/src/compiler/operation-typer.h
[modify] https://crrev.com/b89811227717381e7d50d03cf19fca00f39cc3aa/src/compiler/types.h
[add] https://crrev.com/b89811227717381e7d50d03cf19fca00f39cc3aa/test/mjsunit/regress/regress-crbug-879898.js

Project Member

Comment 59 by bugdroid1@chromium.org, Sep 4

The following revision refers to this bug:
  https://chromium.googlesource.com/v8/v8.git/+/b89811227717381e7d50d03cf19fca00f39cc3aa

commit b89811227717381e7d50d03cf19fca00f39cc3aa
Author: Benedikt Meurer <bmeurer@chromium.org>
Date: Mon Sep 03 19:14:09 2018

[turbofan] Improve typing of ToNumeric and ToNumber.

The previous typing rules for ToNumeric and ToNumber didn't match on the
NonBigIntPrimitive input set, which causes trouble when we morph ToNumeric
nodes into ToNumber nodes, and generally lead to worse typings in the
graph, and thus worse code generation. This change improves the existing
typing rules and turns ToNumber into a chokepoint again.

Bug:  chromium:879898 , v8:8015
Change-Id: I4a7ff0e9c420c5dcfdb2b96884e019a5943828a4
Reviewed-on: https://chromium-review.googlesource.com/1201522
Reviewed-by: Georg Neis <neis@chromium.org>
Commit-Queue: Benedikt Meurer <bmeurer@chromium.org>
Cr-Commit-Position: refs/heads/master@{#55595}
[modify] https://crrev.com/b89811227717381e7d50d03cf19fca00f39cc3aa/src/compiler/operation-typer.cc
[modify] https://crrev.com/b89811227717381e7d50d03cf19fca00f39cc3aa/src/compiler/operation-typer.h
[modify] https://crrev.com/b89811227717381e7d50d03cf19fca00f39cc3aa/src/compiler/types.h
[add] https://crrev.com/b89811227717381e7d50d03cf19fca00f39cc3aa/test/mjsunit/regress/regress-crbug-879898.js

Project Member

Comment 60 by bugdroid1@chromium.org, Sep 4

The following revision refers to this bug:
  https://chromium.googlesource.com/v8/v8.git/+/b89811227717381e7d50d03cf19fca00f39cc3aa

commit b89811227717381e7d50d03cf19fca00f39cc3aa
Author: Benedikt Meurer <bmeurer@chromium.org>
Date: Mon Sep 03 19:14:09 2018

[turbofan] Improve typing of ToNumeric and ToNumber.

The previous typing rules for ToNumeric and ToNumber didn't match on the
NonBigIntPrimitive input set, which causes trouble when we morph ToNumeric
nodes into ToNumber nodes, and generally lead to worse typings in the
graph, and thus worse code generation. This change improves the existing
typing rules and turns ToNumber into a chokepoint again.

Bug:  chromium:879898 , v8:8015
Change-Id: I4a7ff0e9c420c5dcfdb2b96884e019a5943828a4
Reviewed-on: https://chromium-review.googlesource.com/1201522
Reviewed-by: Georg Neis <neis@chromium.org>
Commit-Queue: Benedikt Meurer <bmeurer@chromium.org>
Cr-Commit-Position: refs/heads/master@{#55595}
[modify] https://crrev.com/b89811227717381e7d50d03cf19fca00f39cc3aa/src/compiler/operation-typer.cc
[modify] https://crrev.com/b89811227717381e7d50d03cf19fca00f39cc3aa/src/compiler/operation-typer.h
[modify] https://crrev.com/b89811227717381e7d50d03cf19fca00f39cc3aa/src/compiler/types.h
[add] https://crrev.com/b89811227717381e7d50d03cf19fca00f39cc3aa/test/mjsunit/regress/regress-crbug-879898.js

Project Member

Comment 61 by bugdroid1@chromium.org, Sep 4

The following revision refers to this bug:
  https://chromium.googlesource.com/v8/v8.git/+/b89811227717381e7d50d03cf19fca00f39cc3aa

commit b89811227717381e7d50d03cf19fca00f39cc3aa
Author: Benedikt Meurer <bmeurer@chromium.org>
Date: Mon Sep 03 19:14:09 2018

[turbofan] Improve typing of ToNumeric and ToNumber.

The previous typing rules for ToNumeric and ToNumber didn't match on the
NonBigIntPrimitive input set, which causes trouble when we morph ToNumeric
nodes into ToNumber nodes, and generally lead to worse typings in the
graph, and thus worse code generation. This change improves the existing
typing rules and turns ToNumber into a chokepoint again.

Bug:  chromium:879898 , v8:8015
Change-Id: I4a7ff0e9c420c5dcfdb2b96884e019a5943828a4
Reviewed-on: https://chromium-review.googlesource.com/1201522
Reviewed-by: Georg Neis <neis@chromium.org>
Commit-Queue: Benedikt Meurer <bmeurer@chromium.org>
Cr-Commit-Position: refs/heads/master@{#55595}
[modify] https://crrev.com/b89811227717381e7d50d03cf19fca00f39cc3aa/src/compiler/operation-typer.cc
[modify] https://crrev.com/b89811227717381e7d50d03cf19fca00f39cc3aa/src/compiler/operation-typer.h
[modify] https://crrev.com/b89811227717381e7d50d03cf19fca00f39cc3aa/src/compiler/types.h
[add] https://crrev.com/b89811227717381e7d50d03cf19fca00f39cc3aa/test/mjsunit/regress/regress-crbug-879898.js

Project Member

Comment 62 by bugdroid1@chromium.org, Sep 4

The following revision refers to this bug:
  https://chromium.googlesource.com/v8/v8.git/+/b89811227717381e7d50d03cf19fca00f39cc3aa

commit b89811227717381e7d50d03cf19fca00f39cc3aa
Author: Benedikt Meurer <bmeurer@chromium.org>
Date: Mon Sep 03 19:14:09 2018

[turbofan] Improve typing of ToNumeric and ToNumber.

The previous typing rules for ToNumeric and ToNumber didn't match on the
NonBigIntPrimitive input set, which causes trouble when we morph ToNumeric
nodes into ToNumber nodes, and generally lead to worse typings in the
graph, and thus worse code generation. This change improves the existing
typing rules and turns ToNumber into a chokepoint again.

Bug:  chromium:879898 , v8:8015
Change-Id: I4a7ff0e9c420c5dcfdb2b96884e019a5943828a4
Reviewed-on: https://chromium-review.googlesource.com/1201522
Reviewed-by: Georg Neis <neis@chromium.org>
Commit-Queue: Benedikt Meurer <bmeurer@chromium.org>
Cr-Commit-Position: refs/heads/master@{#55595}
[modify] https://crrev.com/b89811227717381e7d50d03cf19fca00f39cc3aa/src/compiler/operation-typer.cc
[modify] https://crrev.com/b89811227717381e7d50d03cf19fca00f39cc3aa/src/compiler/operation-typer.h
[modify] https://crrev.com/b89811227717381e7d50d03cf19fca00f39cc3aa/src/compiler/types.h
[add] https://crrev.com/b89811227717381e7d50d03cf19fca00f39cc3aa/test/mjsunit/regress/regress-crbug-879898.js

Project Member

Comment 63 by bugdroid1@chromium.org, Sep 4

The following revision refers to this bug:
  https://chromium.googlesource.com/v8/v8.git/+/b89811227717381e7d50d03cf19fca00f39cc3aa

commit b89811227717381e7d50d03cf19fca00f39cc3aa
Author: Benedikt Meurer <bmeurer@chromium.org>
Date: Mon Sep 03 19:14:09 2018

[turbofan] Improve typing of ToNumeric and ToNumber.

The previous typing rules for ToNumeric and ToNumber didn't match on the
NonBigIntPrimitive input set, which causes trouble when we morph ToNumeric
nodes into ToNumber nodes, and generally lead to worse typings in the
graph, and thus worse code generation. This change improves the existing
typing rules and turns ToNumber into a chokepoint again.

Bug:  chromium:879898 , v8:8015
Change-Id: I4a7ff0e9c420c5dcfdb2b96884e019a5943828a4
Reviewed-on: https://chromium-review.googlesource.com/1201522
Reviewed-by: Georg Neis <neis@chromium.org>
Commit-Queue: Benedikt Meurer <bmeurer@chromium.org>
Cr-Commit-Position: refs/heads/master@{#55595}
[modify] https://crrev.com/b89811227717381e7d50d03cf19fca00f39cc3aa/src/compiler/operation-typer.cc
[modify] https://crrev.com/b89811227717381e7d50d03cf19fca00f39cc3aa/src/compiler/operation-typer.h
[modify] https://crrev.com/b89811227717381e7d50d03cf19fca00f39cc3aa/src/compiler/types.h
[add] https://crrev.com/b89811227717381e7d50d03cf19fca00f39cc3aa/test/mjsunit/regress/regress-crbug-879898.js

Project Member

Comment 64 by bugdroid1@chromium.org, Sep 4

The following revision refers to this bug:
  https://chromium.googlesource.com/v8/v8.git/+/b89811227717381e7d50d03cf19fca00f39cc3aa

commit b89811227717381e7d50d03cf19fca00f39cc3aa
Author: Benedikt Meurer <bmeurer@chromium.org>
Date: Mon Sep 03 19:14:09 2018

[turbofan] Improve typing of ToNumeric and ToNumber.

The previous typing rules for ToNumeric and ToNumber didn't match on the
NonBigIntPrimitive input set, which causes trouble when we morph ToNumeric
nodes into ToNumber nodes, and generally lead to worse typings in the
graph, and thus worse code generation. This change improves the existing
typing rules and turns ToNumber into a chokepoint again.

Bug:  chromium:879898 , v8:8015
Change-Id: I4a7ff0e9c420c5dcfdb2b96884e019a5943828a4
Reviewed-on: https://chromium-review.googlesource.com/1201522
Reviewed-by: Georg Neis <neis@chromium.org>
Commit-Queue: Benedikt Meurer <bmeurer@chromium.org>
Cr-Commit-Position: refs/heads/master@{#55595}
[modify] https://crrev.com/b89811227717381e7d50d03cf19fca00f39cc3aa/src/compiler/operation-typer.cc
[modify] https://crrev.com/b89811227717381e7d50d03cf19fca00f39cc3aa/src/compiler/operation-typer.h
[modify] https://crrev.com/b89811227717381e7d50d03cf19fca00f39cc3aa/src/compiler/types.h
[add] https://crrev.com/b89811227717381e7d50d03cf19fca00f39cc3aa/test/mjsunit/regress/regress-crbug-879898.js

Project Member

Comment 65 by ClusterFuzz, Sep 4

ClusterFuzz has detected this issue as fixed in range 55594:55595.

Detailed report: https://clusterfuzz.com/testcase?key=5085453708165120

Fuzzer: ochang_js_fuzzer
Job Type: linux_asan_d8_dbg
Platform Id: linux

Crash Type: CHECK failure
Crash Address: 
Crash State:
  TypeError: node #28:JSToNumber type Numeric is not Number in verifier.cc
  v8::internal::compiler::Verifier::Visitor::CheckTypeIs
  v8::internal::compiler::Verifier::Visitor::Check
  
Sanitizer: address (ASAN)

Regressed: https://clusterfuzz.com/revisions?job=linux_asan_d8_dbg&range=55555:55556
Fixed: https://clusterfuzz.com/revisions?job=linux_asan_d8_dbg&range=55594:55595

Reproducer Testcase: https://clusterfuzz.com/download?testcase_id=5085453708165120

See https://github.com/google/clusterfuzz-tools for more information.

If you suspect that the result above is incorrect, try re-doing that job on the test case report page.
Project Member

Comment 66 by ClusterFuzz, Sep 4

Labels: ClusterFuzz-Verified
Status: Verified (was: Fixed)
ClusterFuzz testcase 5085453708165120 is verified as fixed, so closing issue as verified.

If this is incorrect, please add ClusterFuzz-Wrong label and re-open the issue.
Project Member

Comment 67 by sheriffbot@chromium.org, Sep 4

Labels: -Restrict-View-SecurityTeam Restrict-View-SecurityNotify
Project Member

Comment 68 by bugdroid1@chromium.org, Sep 4

The following revision refers to this bug:
  https://chromium.googlesource.com/v8/v8.git/+/b89811227717381e7d50d03cf19fca00f39cc3aa

commit b89811227717381e7d50d03cf19fca00f39cc3aa
Author: Benedikt Meurer <bmeurer@chromium.org>
Date: Mon Sep 03 19:14:09 2018

[turbofan] Improve typing of ToNumeric and ToNumber.

The previous typing rules for ToNumeric and ToNumber didn't match on the
NonBigIntPrimitive input set, which causes trouble when we morph ToNumeric
nodes into ToNumber nodes, and generally lead to worse typings in the
graph, and thus worse code generation. This change improves the existing
typing rules and turns ToNumber into a chokepoint again.

Bug:  chromium:879898 , v8:8015
Change-Id: I4a7ff0e9c420c5dcfdb2b96884e019a5943828a4
Reviewed-on: https://chromium-review.googlesource.com/1201522
Reviewed-by: Georg Neis <neis@chromium.org>
Commit-Queue: Benedikt Meurer <bmeurer@chromium.org>
Cr-Commit-Position: refs/heads/master@{#55595}
[modify] https://crrev.com/b89811227717381e7d50d03cf19fca00f39cc3aa/src/compiler/operation-typer.cc
[modify] https://crrev.com/b89811227717381e7d50d03cf19fca00f39cc3aa/src/compiler/operation-typer.h
[modify] https://crrev.com/b89811227717381e7d50d03cf19fca00f39cc3aa/src/compiler/types.h
[add] https://crrev.com/b89811227717381e7d50d03cf19fca00f39cc3aa/test/mjsunit/regress/regress-crbug-879898.js

Project Member

Comment 69 by bugdroid1@chromium.org, Sep 4

The following revision refers to this bug:
  https://chromium.googlesource.com/v8/v8.git/+/b89811227717381e7d50d03cf19fca00f39cc3aa

commit b89811227717381e7d50d03cf19fca00f39cc3aa
Author: Benedikt Meurer <bmeurer@chromium.org>
Date: Mon Sep 03 19:14:09 2018

[turbofan] Improve typing of ToNumeric and ToNumber.

The previous typing rules for ToNumeric and ToNumber didn't match on the
NonBigIntPrimitive input set, which causes trouble when we morph ToNumeric
nodes into ToNumber nodes, and generally lead to worse typings in the
graph, and thus worse code generation. This change improves the existing
typing rules and turns ToNumber into a chokepoint again.

Bug:  chromium:879898 , v8:8015
Change-Id: I4a7ff0e9c420c5dcfdb2b96884e019a5943828a4
Reviewed-on: https://chromium-review.googlesource.com/1201522
Reviewed-by: Georg Neis <neis@chromium.org>
Commit-Queue: Benedikt Meurer <bmeurer@chromium.org>
Cr-Commit-Position: refs/heads/master@{#55595}
[modify] https://crrev.com/b89811227717381e7d50d03cf19fca00f39cc3aa/src/compiler/operation-typer.cc
[modify] https://crrev.com/b89811227717381e7d50d03cf19fca00f39cc3aa/src/compiler/operation-typer.h
[modify] https://crrev.com/b89811227717381e7d50d03cf19fca00f39cc3aa/src/compiler/types.h
[add] https://crrev.com/b89811227717381e7d50d03cf19fca00f39cc3aa/test/mjsunit/regress/regress-crbug-879898.js

Project Member

Comment 70 by bugdroid1@chromium.org, Sep 4

The following revision refers to this bug:
  https://chromium.googlesource.com/v8/v8.git/+/b89811227717381e7d50d03cf19fca00f39cc3aa

commit b89811227717381e7d50d03cf19fca00f39cc3aa
Author: Benedikt Meurer <bmeurer@chromium.org>
Date: Mon Sep 03 19:14:09 2018

[turbofan] Improve typing of ToNumeric and ToNumber.

The previous typing rules for ToNumeric and ToNumber didn't match on the
NonBigIntPrimitive input set, which causes trouble when we morph ToNumeric
nodes into ToNumber nodes, and generally lead to worse typings in the
graph, and thus worse code generation. This change improves the existing
typing rules and turns ToNumber into a chokepoint again.

Bug:  chromium:879898 , v8:8015
Change-Id: I4a7ff0e9c420c5dcfdb2b96884e019a5943828a4
Reviewed-on: https://chromium-review.googlesource.com/1201522
Reviewed-by: Georg Neis <neis@chromium.org>
Commit-Queue: Benedikt Meurer <bmeurer@chromium.org>
Cr-Commit-Position: refs/heads/master@{#55595}
[modify] https://crrev.com/b89811227717381e7d50d03cf19fca00f39cc3aa/src/compiler/operation-typer.cc
[modify] https://crrev.com/b89811227717381e7d50d03cf19fca00f39cc3aa/src/compiler/operation-typer.h
[modify] https://crrev.com/b89811227717381e7d50d03cf19fca00f39cc3aa/src/compiler/types.h
[add] https://crrev.com/b89811227717381e7d50d03cf19fca00f39cc3aa/test/mjsunit/regress/regress-crbug-879898.js

Project Member

Comment 71 by sheriffbot@chromium.org, Dec 11

Labels: -Restrict-View-SecurityNotify allpublic
This bug has been closed for more than 14 weeks. Removing security view restrictions.

For more details visit https://www.chromium.org/issue-tracking/autotriage - Your friendly Sheriffbot

Sign in to add a comment