Bug 161756

Summary: REGRESSION (r205569?): 32-bit JSC test timeout: stress/rest-parameter-many-arguments.js.dfg-maximal-flush-validate-no-cjit
Product: WebKit Reporter: Ryan Haddad <ryanhaddad>
Component: JavaScriptCoreAssignee: Mark Lam <mark.lam>
Status: RESOLVED FIXED    
Severity: Normal CC: ggaren, mark.lam
Priority: P2    
Version: WebKit Nightly Build   
Hardware: Unspecified   
OS: Unspecified   
See Also: https://bugs.webkit.org/show_bug.cgi?id=161498
Attachments:
Description Flags
proposed patch. saam: review+

Ryan Haddad
Reported 2016-09-08 13:43:15 PDT
32-bit JSC test timeout: stress/rest-parameter-many-arguments.js.dfg-maximal-flush-validate-no-cjit https://build.webkit.org/builders/Apple%20El%20Capitan%2032-bit%20JSC%20%28BuildAndTest%29/builds/3632
Attachments
proposed patch. (1.21 KB, patch)
2016-09-08 15:00 PDT, Mark Lam 		saam: review+
DetailsFormatted DiffDiff
View All Add attachment proposed patch, testcase, etc.
Ryan Haddad
Comment 1 2016-09-08 13:43:40 PDT
This regressed between r205568 and r205574
Ryan Haddad
Comment 2 2016-09-08 13:44:59 PDT
Possibly https://trac.webkit.org/changeset/205569
Mark Lam
Comment 3 2016-09-08 14:21:01 PDT
Running locally, the test appears to finish just fine.
Mark Lam
Comment 4 2016-09-08 14:53:22 PDT
Looks like the bot is just too slow. Running the full suite locally on MacPro with the test harness, I don't see the time out at all. I'll see if I can skip this test configuration for 32-bit x86.
Mark Lam
Comment 5 2016-09-08 15:00:54 PDT
Created attachment 288333 [details] proposed patch.
Mark Lam
Comment 6 2016-09-08 15:03:31 PDT
Thanks for the review. Landed in r205664: <http://trac.webkit.org/r205664>.
Note You need to log in before you can comment on or make changes to this bug.