Open Bug 1341789 Opened 7 years ago Updated 2 years ago

CacheIR: Don't eagerly emit IR with maybeGuardInt32Index

Categories

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

defect

Tracking

()

Performance Impact none

People

(Reporter: evilpie, 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
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: