Closed
Bug 1087496
Opened 10 years ago
Closed 10 years ago
Optimize IsNullOrLikeUndefined(AndBranch)
Categories
(Core :: JavaScript Engine: JIT, defect)
Core
JavaScript Engine: JIT
Tracking
()
RESOLVED
FIXED
mozilla36
People
(Reporter: jandem, Assigned: jandem)
Details
Attachments
(1 file)
2.50 KB,
patch
|
h4writer
:
review+
|
Details | Diff | Splinter 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 1•10 years ago
|
||
Comment on attachment 8509691 [details] [diff] [review] Patch Review of attachment 8509691 [details] [diff] [review]: ----------------------------------------------------------------- Nice!
Attachment #8509691 -
Flags: review?(hv1989) → review+
Assignee | ||
Comment 2•10 years ago
|
||
Thanks for the quick review! https://hg.mozilla.org/integration/mozilla-inbound/rev/1220cafa6058
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.
Description
•