Bug 184228

Summary:

Raise the for-call inlining threshold to 190 to fix JetStream/richards regression

Product:

WebKit

Reporter:

Filip Pizlo <fpizlo>

Component:

JavaScriptCore

Assignee:

Filip Pizlo <fpizlo>

Status:

RESOLVED FIXED

Severity:

Normal

CC:

fpizlo, jfbastien, keith_miller, mark.lam, msaboff, rmorisset, saam, webkit-bug-importer, ysuzuki

Priority:

Keywords:

InRadar

Version:

WebKit Nightly Build

Hardware:

All

OS:

All

Attachments:

Description	Flags
the patch	ysuzuki: review+

Description Filip Pizlo 2018-04-01 12:02:35 PDT

I regressed JetStream/richards in https://trac.webkit.org/changeset/230115/webkit.  This fixes it.

Comment 1 Filip Pizlo 2018-04-01 12:03:31 PDT

Created attachment 336959 [details]
the patch

Comment 2 Yusuke Suzuki 2018-04-01 12:09:24 PDT

Comment on attachment 336959 [details]
the patch

r=me if we do not get regressions.

Comment 3 Filip Pizlo 2018-04-01 12:13:10 PDT

Local testing says it’s a progression! :-)

Comment 4 Filip Pizlo 2018-04-01 12:46:21 PDT

Landed in https://trac.webkit.org/changeset/230145/webkit

Comment 5 Radar WebKit Bug Importer 2018-04-01 12:47:24 PDT

<rdar://problem/39089329>