Closed
Bug 1041418
Opened 10 years ago
Closed 10 years ago
Update Shumway's JS optimization for TypeScript
Categories
(Firefox Graveyard :: Shumway, defect)
Firefox Graveyard
Shumway
Tracking
(Not tracked)
RESOLVED
FIXED
People
(Reporter: cpeterson, Assigned: tschneider)
References
Details
(Keywords: perf)
The master branch currently has two JS optimization passes:
1. Remove `if (DEBUG)` code in release builds with Esprima compiler
2. Optimize JS code with Closure compiler
We need to enable these optimizations on the nat branch. Till says Closure does not able to optimize TypeScripts' generated JS very well. We might need to investigate other optimizers.
Reporter | ||
Comment 1•10 years ago
|
||
We should also evaluate the TypeScript compiler:
http://blogs.msdn.com/b/typescript/archive/2014/07/21/new-compiler-and-moving-to-github.aspx
Summary: Update Shumway's JS optimization passes for TypeScript → Update Shumway's JS optimization for TypeScript
Updated•10 years ago
|
Assignee: nobody → mbebenita
Status: NEW → ASSIGNED
Comment 2•10 years ago
|
||
I should mention that the TS compiler team added an optimization pass to remove unnecessary rest arguments. These were causing lots of allocations in the JS engine, since they couldn't be optimized away at the JS level.
https://github.com/Microsoft/TypeScript/issues/498
In other news, I've already added closure support to the build system. This will constant fold all references to `debug` or `release`, but hasn't been tested, yet.
I'll need info yury for now to see if it can be integrated into the build system.
Flags: needinfo?(ydelendik)
Comment 3•10 years ago
|
||
Since Tobias is assigned to the blocking bug, I'll reassign this also.
Assignee: mbebenita → schneider
Flags: needinfo?(ydelendik) → needinfo?(schneider)
Assignee | ||
Comment 4•10 years ago
|
||
Taken. See updates on https://bugzilla.mozilla.org/show_bug.cgi?id=1063367.
Assignee | ||
Updated•10 years ago
|
Flags: needinfo?(schneider)
Reporter | ||
Comment 5•10 years ago
|
||
Tobias: was this bug fixed by bug 1063367?
Flags: needinfo?(schneider)
Assignee | ||
Updated•10 years ago
|
Status: ASSIGNED → RESOLVED
Closed: 10 years ago
Flags: needinfo?(schneider)
Resolution: --- → FIXED
Updated•9 years ago
|
Product: Firefox → Firefox Graveyard
You need to log in
before you can comment on or make changes to this bug.
Description
•