Open Bug 565534 Opened 9 years ago Updated 5 years ago

Select JSOP_POP not JSOP_POPV for non-tail expression statements

Categories

(Core :: JavaScript Engine, enhancement, P3)

enhancement

Tracking

()

ASSIGNED

People

(Reporter: brendan, Assigned: brendan)

References

(Depends on 1 open bug)

Details

Attachments

(1 file)

Attached patch proposed fixSplinter Review
Seems easy enough. Perf results in a few.

/be
Attachment #445019 - Flags: review?(mrbkap)
Perf change is noise.

/be
So, I seem to recall the tail call proposal in harmony having a complex definition of what consists of "tail position", and that it involved determining whether you're in a catch block, etc. Do we not have to do that here?
(In reply to comment #2)
> So, I seem to recall the tail call proposal in harmony having a complex
> definition of what consists of "tail position", and that it involved
> determining whether you're in a catch block, etc. Do we not have to do that
> here?

Done in patch for bug 565374, which blocks this and on which bug's patch this bug's patch is based (notice use of pn_tailpos, which is not in tm tip yet!). See the URL for bug 565374 which links to the ES4-era spec.

/be
Attachment #445019 - Flags: review?(mrbkap) → review+
Target Milestone: mozilla1.9.3a5 → ---
Blake: is Brendan's r+'d patch (from 2010) still relevant?
Flags: needinfo?(mrbkap)
Not really. The code has changed dramatically since this patch was written (and we'd have to land bug 565374 first anyway).
Flags: needinfo?(mrbkap)
You need to log in before you can comment on or make changes to this bug.