Closed Bug 1647370 Opened 2 months ago Closed 2 months ago

Known JSClass analysis based on MIR

Categories

(Core :: JavaScript Engine: JIT, task, P1)

Product:
Core
Component:
JavaScript Engine: JIT
Type:
task

Tracking

()

Status:
RESOLVED FIXED
Milestone:
mozilla79

People

(Reporter: evilpie, Assigned: evilpie)

References

(Blocks 1 open bug)

Details

Attachments

(1 file, 1 obsolete file)

Bug 1647370 - Known JSClass analysis based on MIR nodes. r?jandem
47 bytes, text/x-phabricator-request
Details | Review

Without TI we don't really have a way of folding instructions based on the input's JSClass. Combined with Warp actually adding more guards instructions this approach can eliminate some trivial guards like GuardToClass = Array for array.length, where array is an array literal. This optimization is only applicable to the current MIR graph. With inlining this of course might become more useful, because e.g. many self-hosted functions have class checks.

Attachment #9158264 - Attachment description: Bug 1647370 - [WIP] Known JSClass analysis based on MIR → Bug 1647370 - Known JSClass analysis based on MIR nodes. r?jandem
Severity: -- → N/A
Priority: -- → P1
Pushed by evilpies@gmail.com:
https://hg.mozilla.org/integration/autoland/rev/e3fd7247646f
Known JSClass analysis based on MIR nodes. r=jandem
Keywords: leave-open
Blocks: 1649398

Comment on attachment 9159449 [details]
Bug 1647370 - Known class support for MRegExp and folding for MHasClass and MIsCallable. r?jandem

Revision D81179 was moved to bug 1649398. Setting attachment 9159449 [details] to obsolete.

Attachment #9159449 - Attachment is obsolete: true
Status: NEW → RESOLVED
Closed: 2 months ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla79
You need to log in before you can comment on or make changes to this bug.