Bug 141123

Summary: Create tests for JSC's Control Flow Profiler
Product: WebKit Reporter: Saam Barati <saam>
Component: JavaScriptCoreAssignee: Saam Barati <saam>
Status: RESOLVED FIXED    
Severity: Normal CC: fpizlo, ggaren
Priority: P2    
Version: 528+ (Nightly build)   
Hardware: Unspecified   
OS: Unspecified   
Attachments:
Description Flags
patch
none
patch fpizlo: review+

Saam Barati
Reported 2015-01-31 12:00:50 PST
There are currently not tests for this and it's time to change that.
Attachments
patch (17.46 KB, patch)
2015-01-31 16:51 PST, Saam Barati 		no flags
DetailsFormatted DiffDiff
patch (17.46 KB, patch)
2015-02-01 19:38 PST, Saam Barati 		fpizlo: review+
DetailsFormatted DiffDiff
Show Obsolete (1) View All Add attachment proposed patch, testcase, etc.
Saam Barati
Comment 1 2015-01-31 16:51:48 PST
Created attachment 245809 [details] patch added preliminary tests. More to come in a later patch.
Saam Barati
Comment 2 2015-02-01 19:38:47 PST
Created attachment 245853 [details] patch See if Win build is fixed.
Filip Pizlo
Comment 3 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.
Saam Barati
Comment 4 2015-02-02 11:06:19 PST
landed in: http://trac.webkit.org/changeset/179479
Note You need to log in before you can comment on or make changes to this bug.