164227 2016-10-31 09:46:48 -0700 Debug JSC test microbenchmarks/pure-get-by-id-cse-2.js timing out 2016-11-15 12:10:22 -0800 1 1 1 Unclassified WebKit JavaScriptCore Other Unspecified Unspecified RESOLVED FIXED https://bugs.webkit.org/show_bug.cgi?id=163305 P2 Normal --- 1 ryanhaddad saam cdumez saam oldest_to_newest 1246467 0 ryanhaddad 2016-10-31 09:46:48 -0700 ** The following JSC stress test failures have been introduced: microbenchmarks/pure-get-by-id-cse-2.js.dfg-eager microbenchmarks/pure-get-by-id-cse-2.js.dfg-eager-no-cjit-validate microbenchmarks/pure-get-by-id-cse-2.js.dfg-maximal-flush-validate-no-cjit microbenchmarks/pure-get-by-id-cse-2.js.no-cjit-validate-phases microbenchmarks/pure-get-by-id-cse-2.js.no-ftl microbenchmarks/pure-get-by-id-cse-2.js.no-llint https://build.webkit.org/builders/Apple%20El%20Capitan%20Debug%20JSC%20%28Tests%29/builds/4409 1246475 1 saam 2016-10-31 10:06:55 -0700 Will fix now. 1246480 2 saam 2016-10-31 10:29:50 -0700 landed fix in: https://trac.webkit.org/changeset/208160 1250244 3 cdumez 2016-11-11 08:54:10 -0800 Reverted r208117 and r208160 for reason: Regressed Speedometer by >1.5% Committed r208588: <http://trac.webkit.org/changeset/208588> 1251542 4 ap 2016-11-15 12:03:43 -0800 Looks like both revisions got reverted, so this must fixed for good. Is that correct? 1251549 5 saam 2016-11-15 12:10:22 -0800 (In reply to comment #4) > Looks like both revisions got reverted, so this must fixed for good. Is that > correct? Yeah sounds correct to me. I'm going to try to re-land the patch at some point soon, so I'll make sure to make this test run for less time when I do.