Closed Bug 998170 Opened 6 years ago Closed 6 years ago

Eliminate LPhi's numInputs_ member

Categories

(Core :: JavaScript Engine: JIT, enhancement)

enhancement
Not set

Tracking

()

RESOLVED FIXED
mozilla31

People

(Reporter: sunfish, Assigned: sunfish)

Details

Attachments

(1 file)

LPhi currently contains a numInputs_ field. This is redundant, since it's always the same as the MPhi's numOperands() (not to mention the block's numPredecessors()). And it isn't used all that often.
Attached is a patch which removes LPhi's numOperands_ member. It also changes some places which call LPhi's numOperands() to call it somewhat less often, now that it's slightly slower.
Assignee: nobody → sunfish
Attachment #8408729 - Flags: review?(hv1989)
Attachment #8408729 - Flags: review?(hv1989) → review+
https://hg.mozilla.org/mozilla-central/rev/e3587048a9fd
Status: NEW → RESOLVED
Closed: 6 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla31
You need to log in before you can comment on or make changes to this bug.