Bug 142946

Summary:

tests/stress/*tdz* tests do 10x more iterations than necessary

Product:

WebKit

Reporter:

Filip Pizlo <fpizlo>

Component:

JavaScriptCore

Assignee:

Filip Pizlo <fpizlo>

Status:

RESOLVED FIXED

Severity:

Normal

CC:

rniwa

Priority:

Version:

528+ (Nightly build)

Hardware:

All

OS:

All

Attachments:

Description	Flags
the patch	none
the patch	rniwa: review+

Description Filip Pizlo 2015-03-22 10:08:03 PDT

Stress tests should run for 10,000 iterations, where each iteration calls a function marked noInline, and that function contains the actual test content. Running for 100,000 iterations is 10x too many.

Comment 1 Filip Pizlo 2015-03-22 10:09:50 PDT

Created attachment 249199 [details]
the patch

Comment 2 Filip Pizlo 2015-03-22 11:21:18 PDT

Created attachment 249201 [details]
the patch

Added some commentary.

Comment 3 Filip Pizlo 2015-03-22 11:26:26 PDT

Landed in http://trac.webkit.org/changeset/181833