Open Bug 1341789 Opened 9 years ago Updated 7 months ago

CacheIR: Don't eagerly emit IR with maybeGuardInt32Index

Categories

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

defect

Tracking

()

Performance Impact none

People

(Reporter: evilpies, Unassigned)

References

(Blocks 2 open bugs)

Details

Attachments

(1 file)

Currently maybeGuardInt32Index will always emit a index guard when called. This is too restrictive. For example currently we can't move tryAttachProxyElement after tryAttachTypedElement. This also prevents us from optimizing indexing with a string into primitive strings or magic arguments.
Attached patch WIPSplinter Review
I tried this out before, but I don't like the way it turned out much. I am not sure arguments we should pass to the tryAttachXXX functions.
Whiteboard: [qf-]
Priority: -- → P3
Performance Impact: --- → -
Whiteboard: [qf-]
Severity: normal → S3

Still relevant?

You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: