Bug 141123

Summary:

Create tests for JSC's Control Flow Profiler

Product:

WebKit

Reporter:

Saam Barati <saam>

Component:

JavaScriptCore

Assignee:

Saam Barati <saam>

Status:

RESOLVED FIXED

Severity:

Normal

CC:

fpizlo, ggaren

Priority:

Version:

528+ (Nightly build)

Hardware:

Unspecified

OS:

Unspecified

Attachments:

Description	Flags
patch	none
patch	fpizlo: review+

Description Saam Barati 2015-01-31 12:00:50 PST

There are currently not tests for this and it's time to change that.

Comment 1 Saam Barati 2015-01-31 16:51:48 PST

Created attachment 245809 [details]
patch

added preliminary tests. More to come in a later patch.

Comment 2 Saam Barati 2015-02-01 19:38:47 PST

Created attachment 245853 [details]
patch

See if Win build is fixed.

Comment 3 Filip Pizlo 2015-02-02 10:53:55 PST

Comment on attachment 245853 [details]
patch

View in context: https://bugs.webkit.org/attachment.cgi?id=245853&action=review

> Tools/Scripts/run-jsc-stress-tests:738
> +       run("no-cjit-type-profiler", "--enableControlFlowProfiler=true", *NO_CJIT_OPTIONS)

This needs one more space of indentation.

Comment 4 Saam Barati 2015-02-02 11:06:19 PST

landed in:
http://trac.webkit.org/changeset/179479