Closed Bug 891400 Opened 9 years ago Closed 9 years ago
Monkey: Pattern match better on static typed array indexes
Bug 864214 added pattern matching so that static typed array accesses can be generated, but the pattern matching done on the typed array index does not capture all cases. Namely, no shift is needed if the array has an element width of one, and accesses at constant indexes are not handled. This was causing 30% of typed array accesses in asm.js-zlib to use the generic path, which are fixed by this patch.
Attachment #772687 - Flags: review?(jdemooij)
Attachment #772687 - Flags: review?(jdemooij) → review+
Followup fix. This exposed a preexisting issue where LoadTypedArrayElementStatic never does bounds checks when loading from uint32 arrays. https://hg.mozilla.org/integration/mozilla-inbound/rev/16e818a58357
Status: NEW → RESOLVED
Closed: 9 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla25
Depends on: 894727
Depends on: 893732
Depends on: 897202
No longer depends on: 897202
Depends on: 899510
You need to log in before you can comment on or make changes to this bug.