Closed Bug 1087496 Opened 10 years ago Closed 10 years ago

Optimize IsNullOrLikeUndefined(AndBranch)

Categories

(Core :: JavaScript Engine: JIT, defect)

defect
Not set
normal

Tracking

()

RESOLVED FIXED
mozilla36

People

(Reporter: jandem, Assigned: jandem)

Details

Attachments

(1 file)

Attached patch PatchSplinter Review
These instructions currently emit two branches, to handle null and undefined. In many cases though we know the input can be null but not undefined, or vice versa, so we have to emit only one branch.

Octane-richards has some hot IsNullOrLikeUndefinedAndBranch tests, where we know the input is never undefined.

There's probably more room for improvement here but we can do that later.
Attachment #8509691 - Flags: review?(hv1989)
Comment on attachment 8509691 [details] [diff] [review]
Patch

Review of attachment 8509691 [details] [diff] [review]:
-----------------------------------------------------------------

Nice!
Attachment #8509691 - Flags: review?(hv1989) → review+
https://hg.mozilla.org/mozilla-central/rev/1220cafa6058
Status: ASSIGNED → RESOLVED
Closed: 10 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla36
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: