1501316 - Cleanup TableSwitch handling in Baseline to no longer use an IC

Status: RESOLVED FIXED

(Core :: JavaScript Engine: JIT, enhancement, P3)

Description

[Ted Campbell [:tcampbell]]

Currently the TableSwitchIC is doing what we would otherwise do directly in the mainline code. It seems this was done for memory savings. Another approach would be to treat this like our other helper-stub / trampolines.

At the same time, we should look at moving the address tables to the BaselineScript itself since they should be well known during compile. This brings them more aligned with our yeildAndAwait table.

Comment 1

[Jan de Mooij [:jandem]]

We need this for bug 1499644 :)

Comment 2

[Jan de Mooij [:jandem]]

Attachment: Bug 1501316 - Remove Baseline's TableSwitch IC, use resume entries instead. r?tcampbell

To not affect Baseline code size too much, this adds a trampoline for the
double-to-int32 Value conversion.

Depends on D11019

Comment 3

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/7283b13fd2d2
Remove Baseline's TableSwitch IC, use resume entries instead. r=tcampbell

Comment 4

[Dorel Luca [:dluca]]

https://hg.mozilla.org/mozilla-central/rev/7283b13fd2d2