Consider removing frame annotations

RESOLVED FIXED in mozilla21

Status

()

defect
RESOLVED FIXED
6 years ago
6 years ago

People

(Reporter: jandem, Assigned: jandem)

Tracking

unspecified
mozilla21
Points:
---

Firefox Tracking Flags

(Not tracked)

Details

Attachments

(1 attachment)

(Assignee)

Description

6 years ago
bholley says he removed the last use of frame annotations in Gecko a while ago. Frame annotations complicate baseline compiler work, so it would be nice if we could remove them.

David, Luke, any thoughts on this, especially from a JSAPI point-of-view?
Fire ze missiles!  We've been wanting to do this forever (bug 546848).
(Assignee)

Comment 2

6 years ago
(In reply to Luke Wagner [:luke] from comment #1)
> Fire ze missiles!  We've been wanting to do this forever (bug 546848).

Great! Taking.
Assignee: general → jdemooij
Status: NEW → ASSIGNED
(Assignee)

Comment 3

6 years ago
Posted patch PatchSplinter Review
Attachment #704590 - Flags: review?(luke)
Comment on attachment 704590 [details] [diff] [review]
Patch

r+ with prejudice (and with much love for bholley).
Attachment #704590 - Flags: review?(luke) → review+
(In reply to Luke Wagner [:luke] from comment #4)
> r+ with prejudice (and with much love for bholley).

<3

Comment 8

6 years ago
https://hg.mozilla.org/mozilla-central/rev/8116ecda192a
Status: ASSIGNED → RESOLVED
Last Resolved: 6 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla21
You need to log in before you can comment on or make changes to this bug.