The default bug view has changed. See this FAQ.

Make classes work in the JITs.

NEW
Assigned to

Status

()

Core
JavaScript Engine: JIT
P1
normal
2 years ago
4 days ago

People

(Reporter: efaust, Assigned: tcampbell)

Tracking

(Depends on: 3 bugs, Blocks: 4 bugs, {perf})

Firefox Tracking Flags

(Not tracked)

Details

(Reporter)

Description

2 years ago
At present, we have classes, but none of the ops are implemented in the jits. We can at least write simple stubs to solve this problem.
(Reporter)

Updated

2 years ago
Depends on: 1169743
(Reporter)

Updated

2 years ago
Depends on: 1169745
(Reporter)

Updated

2 years ago
Depends on: 1169746
Blocks: 1298286

Updated

6 months ago
Blocks: 1307395
Blocks: 1307062
Priority: -- → P2
Keywords: perf
Assignee: efaustbmo → nobody
Status: ASSIGNED → NEW
Priority: P2 → P1
Realistically this will be for next release.
Priority: P1 → P2
Priority: P2 → P1
(Assignee)

Updated

a month ago
Assignee: nobody → tcampbell
Is there any progress on this bug?

How bad would be the performance if we're going to use `class` instead of `function` and `prototype` in firefox codebase? We already have them in WebExt, and I'm tempted to use them in our devtools, but not if we got some perf issues.
Flags: needinfo?(tcampbell)
(In reply to Matteo Ferretti [:zer0] [:matteo] from comment #2)
> Is there any progress on this bug?
> 
> How bad would be the performance if we're going to use `class` instead of
> `function` and `prototype` in firefox codebase? We already have them in
> WebExt, and I'm tempted to use them in our devtools, but not if we got some
> perf issues.

For now I would say to not use it in performance critical code. Ted is going to look into implementing this in this or next release. He is finishing another bug first.
(In reply to Hannes Verschore [:h4writer] from comment #3)

> For now I would say to not use it in performance critical code. Ted is going
> to look into implementing this in this or next release. He is finishing
> another bug first.

Thanks, it's as I thought. I'll keep on eye on the progress of this bug to understand when classes could be used without perf issues.
Flags: needinfo?(tcampbell)
(Assignee)

Comment 5

5 days ago
Yep, it's next on my list.
You need to log in before you can comment on or make changes to this bug.