Issue metadata
Sign in to add a comment
|
Performance impact of shipping try/catch in TurboFan |
||||||||||||||||||||||
Issue descriptionSee the link to graphs below.
,
Apr 13 2016
...and on chromium-rel-mac10.
,
Apr 14 2016
,
Apr 19 2016
Minimum range for regression is only two changes: 386659 or 386660. The main suspect is: commit 0a263edeb740bb85eebfe9df27ba5870db1801b6 author v8-autoroll <v8-autoroll@chromium.org> Tue Apr 12 12:34:26 2016 committer Commit bot <commit-bot@chromium.org> Tue Apr 12 12:35:54 2016 Update V8 to version 5.2.10.1 (cherry-pick). Assigning to OWNER for further triage...
,
Apr 20 2016
tkent@, could you advise? When the suspect is v8-autoroll, who should be the owner?
,
Apr 20 2016
Assign hablich@ for triage.
,
Apr 20 2016
,
Apr 20 2016
Suspecting https://chromium.googlesource.com/v8/v8/+/4dfc6254fb91b0d058f4c10bd88c897833826f3a (ship try/catch for TF). Jaro & Michael WDYT?
,
Apr 20 2016
,
Apr 21 2016
,
Apr 21 2016
,
Apr 21 2016
Just for completeness: We regress because of Turbofan compilation time - Turbofan is orders of magnitude slower than fullcode. This CL just uses Turbofan for more code.
,
Apr 21 2016
,
Jun 1 2016
Moving this nonessential bug to the next milestone. For more details visit https://www.chromium.org/issue-tracking/autotriage - Your friendly Sheriffbot
,
Jul 13 2016
This issue has been moved once and is lower than Pri-1. Removing the milestone. For more details visit https://www.chromium.org/issue-tracking/autotriage - Your friendly Sheriffbot
,
Oct 11 2016
Perf sheriff ping
,
Oct 17 2016
Looks like this was a tradeoff that we accepted -- jarin@, is there anything else that we want to do here, or should this bug be closed?
,
Oct 18 2016
Yes, we took the trade-off. Closing. |
|||||||||||||||||||||||
►
Sign in to add a comment |
|||||||||||||||||||||||
Comment 1 by m...@chromium.org
, Apr 13 2016