Closed
Bug 1064777
Opened 10 years ago
Closed 10 years ago
Remove unnecessary CNG checks in some places
Categories
(Core :: JavaScript Engine, defect)
Core
JavaScript Engine
Tracking
()
RESOLVED
FIXED
mozilla35
People
(Reporter: jandem, Assigned: jandem)
References
Details
Attachments
(2 files)
1.11 KB,
patch
|
bhackett1024
:
review+
|
Details | Diff | Splinter Review |
1.76 KB,
patch
|
bhackett1024
:
review+
|
Details | Diff | Splinter Review |
After enabling Ion/TI everywhere (bug 939562) and Ion-compiling non-CNG functions (bug 1045529), there are still some performance problems in non-CNG (chrome) code caused by (unnecessary) CNG checks in various places. I'll post some patches to remove them, so that we can hopefully close the gap between content and chrome even more.
Assignee | ||
Comment 1•10 years ago
|
||
This makes the addon in bug 911570 a lot faster.
Attachment #8486320 -
Flags: review?(bhackett1024)
Updated•10 years ago
|
Attachment #8486320 -
Flags: review?(bhackett1024) → review+
Assignee | ||
Comment 2•10 years ago
|
||
Part 1: https://hg.mozilla.org/integration/mozilla-inbound/rev/62e5db38c251
Keywords: leave-open
Comment 3•10 years ago
|
||
https://hg.mozilla.org/mozilla-central/rev/62e5db38c251
Assignee | ||
Comment 4•10 years ago
|
||
Attachment #8487820 -
Flags: review?(bhackett1024)
Updated•10 years ago
|
Attachment #8487820 -
Flags: review?(bhackett1024) → review+
Assignee | ||
Comment 5•10 years ago
|
||
Part 2: https://hg.mozilla.org/integration/mozilla-inbound/rev/d4b3b9593d01
Keywords: leave-open
Comment 6•10 years ago
|
||
https://hg.mozilla.org/mozilla-central/rev/d4b3b9593d01
Status: ASSIGNED → RESOLVED
Closed: 10 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla35
You need to log in
before you can comment on or make changes to this bug.
Description
•