Closed Bug 765733 Opened 12 years ago Closed 12 years ago

Don't emit singleton array initializers for empty arrays

Categories

(Core :: JavaScript Engine, defect)

defect
Not set
normal

Tracking

()

RESOLVED FIXED
mozilla16

People

(Reporter: jandem, Assigned: jandem)

References

(Blocks 1 open bug)

Details

(Whiteboard: [js:t])

Attachments

(1 file)

Attached patch PatchSplinter Review
Kraken crypto-aes/crypto-ccm have a bunch of arrays like [[], [], [], []]. The inner/empty arrays have "unknown properties" because fixArrayType cannot easily determine their type.

This is at least a 10% Kraken crypto-aes/crypto-ccm IonMonkey win since it avoids some GETELEM/SETELEM IC's. JM+TI win is smaller.
Attachment #634035 - Flags: review?(bhackett1024)
Attachment #634035 - Flags: review?(bhackett1024) → review+
Whiteboard: [js:t]
https://hg.mozilla.org/mozilla-central/rev/e7d593a7e561
Status: ASSIGNED → RESOLVED
Closed: 12 years ago
Resolution: --- → FIXED
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: