Closed Bug 1567157 Opened 5 years ago Closed 4 years ago

JIT simplifications after JSScript/LazyScript unification

Categories

(Core :: JavaScript Engine, enhancement, P2)

enhancement

Tracking

()

RESOLVED FIXED
mozilla76
Tracking Status
firefox76 --- fixed

People

(Reporter: tcampbell, Assigned: tcampbell)

References

Details

Attachments

(2 files)

A number of comments and helper functions can be simplified in IonMonkey now that LazyScript has a jitCodeRaw_. See review comments on https://phabricator.services.mozilla.com/D38255

Blocks: 1529456

Re-purposing this bug a bit...

We should also fix MacroAssembler::loadJitCodeMaybeNoArgCheck which no longer needs to worry about lambda clones being lazy when the template is not. This method no longer needs to be "Maybe".

No longer blocks: 1566607
Summary: Ion cleanups now that LazyScript is a JitEntry → JIT simplifications after JSScript/LazyScript unification

These comments refer to concerns from when lambda clones might still be
pointing to a lazy script. Now, the ICs will be tossed before relazification
happens and if we observed any clone having bytecode, then they all will
agree.

Ion only uses this entry for known targets, but previously had to worry about
lambda clones still having a pointer to lazy script. With unified scripts, we
know all clones of same target will have bytecode. On relazification the Ion
code will be tossed as well.

Depends on D69206

Pushed by tcampbell@mozilla.com:
https://hg.mozilla.org/integration/autoland/rev/eebbf5a0f70e
Remove stale jit comments about relazification. r=jandem
https://hg.mozilla.org/integration/autoland/rev/beb1529f957d
Make no-arg-check JIT entry unconditional. r=jandem
Status: ASSIGNED → RESOLVED
Closed: 4 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla76
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: