Closed Bug 1177139 Opened 9 years ago Closed 7 years ago

Apply effective address and alignment mask analyses to M{Load,Store}TypedArrayElementStatic

Categories

(Core :: JavaScript Engine: JIT, defect, P5)

defect

Tracking

()

RESOLVED WONTFIX
Tracking Status
firefox41 --- affected

People

(Reporter: bhackett1024, Assigned: bhackett1024)

References

(Blocks 1 open bug)

Details

Attachments

(1 file)

Attached patch patchSplinter 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: nobody → bhackett1024
Attachment #8625799 - Flags: review?(luke) → review+
Brian, can we land this?
Flags: needinfo?(bhackett1024)
Priority: -- → P5
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: 7 years ago
Flags: needinfo?(bhackett1024)
Resolution: --- → WONTFIX
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: