Closed Bug 837679 Opened 11 years ago Closed 11 years ago

BaselineCompiler: Compile JSOP_TOID

Categories

(Core :: JavaScript Engine, defect)

defect
Not set
normal

Tracking

()

RESOLVED FIXED

People

(Reporter: jandem, Assigned: jandem)

References

Details

Attachments

(1 file)

      No description provided.
Attached patch PatchSplinter Review
JSOP_TOID is used for expressions like obj[idx]++, to avoid converting idx twice (observable if idx is an object). v8-crypto uses it but other than that it's not very common.

JSOP_TOID does not need its own IC, the only case IonMonkey cares about is if the input is int32, in that case TOID is a no-op. If idx is not known to be int32, Ion falls back to a VM call.
Attachment #709729 - Flags: review?(kvijayan)
Attachment #709729 - Flags: review?(kvijayan) → review+
https://hg.mozilla.org/projects/ionmonkey/rev/419f7cb7d3f4
Status: ASSIGNED → RESOLVED
Closed: 11 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: