Closed Bug 520809 Opened 16 years ago Closed 16 years ago

optimize late bound indexed property access, i.e. a[i]

Categories

(Tamarin Graveyard :: Virtual Machine, defect, P5)

defect

Tracking

(Not tracked)

VERIFIED FIXED
flash10.1

People

(Reporter: edwsmith, Assigned: edwsmith)

References

Details

Attachments

(2 obsolete files)

affected opcodes: OP_setproperty OP_initproperty OP_getproperty phase 1: remove boilerplate from jit code, have helpers handle the common cases phase 2: (maybe) use inline caching also, add common-case handlers keyed on object type (typically Array) and index type (typically int)
Assignee: nobody → edwsmith
Attachment #404889 - Flags: review?(stejohns)
this one is for OP_getproperty, adds getprop_index() helper and removes boilerplate around the most-generic slow case.
Attachment #404891 - Flags: review?(stejohns)
Attachment #404889 - Flags: review?(stejohns) → review+
Attachment #404891 - Flags: review?(stejohns) → review+
Priority: -- → P5
Target Milestone: --- → flash10.1
Comment on attachment 404889 [details] [diff] [review] adds optimized helper for OP_setproperty and OP_initproperty with indexed multiname pushed http://hg.mozilla.org/tamarin-redux/rev/b52b706ec82d
Attachment #404889 - Attachment is obsolete: true
Comment on attachment 404891 [details] [diff] [review] optimized helper for OP_getproperty with index http://hg.mozilla.org/tamarin-redux/rev/c22fc978ac9e
Attachment #404891 - Attachment is obsolete: true
OS: Mac OS X → All
Hardware: x86 → All
Status: NEW → RESOLVED
Closed: 16 years ago
Resolution: --- → FIXED
Engineering work item. Marking as verified.
Status: RESOLVED → VERIFIED
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Creator:
Created:
Updated:
Size: