Closed Bug 608782 Opened 14 years ago Closed 11 years ago

JM: inline fast path for string concatenation

Categories

(Core :: JavaScript Engine, defect)

Product:
Core
Component:
JavaScript Engine
Type:
defect
Priority:
Not set
Severity:
normal

Tracking

()

Status:
RESOLVED WONTFIX

People

(Reporter: luke, Unassigned)

References

Details

(Even after the infallible-malloc hack in bug 608776) callgrind shows 30% of date-format-xparb is spent below stubs::Add.  8% is spent in stubs::Add itself.  With bug 608776 taking the malloc out of js_ConcatStrings (usually), we should be able to write a stub-call-free fast path which either creates a rope node or fills a short string.  Because xparb uses String objects (costing, even after bug 603017, 5% in js::DefaultValue), this fast path should also include an IC for 'toString' on objects.
Depends on: 608776
Blocks: 609212
I think bug 609440 is going ahead instead of bug 608776.  The net effect w.r.t. this bug should be the same, though.
Depends on: 609440
No longer depends on: 608776
Throwing back in the pool.
Assignee: luke → general
Whiteboard: [good first bug]
un-good-first-bugging.  With IonMonkey development full steam, this should either be WONTFIX'd or re-targeted towards IonMonkey.
Whiteboard: [good first bug]
Too late, JM was removed.
Status: ASSIGNED → RESOLVED
Closed: 11 years ago
Resolution: --- → WONTFIX
You need to log in before you can comment on or make changes to this bug.