Closed
Bug 1177139
Opened 10 years ago
Closed 8 years ago
Apply effective address and alignment mask analyses to M{Load,Store}TypedArrayElementStatic
Categories
(Core :: JavaScript Engine: JIT, defect, P5)
Core
JavaScript Engine: JIT
Tracking
()
RESOLVED
WONTFIX
| Tracking | Status | |
|---|---|---|
| firefox41 | --- | affected |
People
(Reporter: bhackett1024, Assigned: bhackett1024)
References
(Blocks 1 open bug)
Details
Attachments
(1 file)
|
14.16 KB,
patch
|
luke
:
review+
|
Details | Diff | Splinter Review |
The effective address and alignment mask analyses look for MAsmJS{Load,Store}Heap accesses to optimize. These analyses can also be used on M{Load,Store}TypedArrayElementStatic, which are similar instructions that are emitted by the normal Ion frontend. Making this change removes most of the perf difference between --asmjs and --no-asmjs for me on the awfy zlib-throughput benchmark on x86.
Attachment #8625799 -
Flags: review?(luke)
|
Assignee | |
Updated•10 years ago
|
Assignee: nobody → bhackett1024
|
||
Updated•10 years ago
|
Attachment #8625799 -
Flags: review?(luke) → review+
|
|
Assignee | |
Comment 2•8 years ago
|
||
Hmm, this patch does not apply cleanly (or at all, really) and probably isn't worth trying to update and apply (things have changed a little in the last year and a half).
Status: NEW → RESOLVED
Closed: 8 years ago
Flags: needinfo?(bhackett1024)
Resolution: --- → WONTFIX
You need to log in
before you can comment on or make changes to this bug.
Description
•