1604467 - Remove SRC_TRY source note

Status: RESOLVED FIXED

(Core :: JavaScript Engine, task, P2)

Description

[Jan de Mooij [:jandem]]

This is the only source note still used by the JITs (BytecodeAnalysis and IonBuilder). I think this after-try offset should be stored as JSOP_TRY operand instead and then JIT compilation never needs to look up source notes, which seems nice.

We can then remove IonBuilder::gsn and the gsnCache in BytecodeAnalysis::init.

Comment 1

[Jan de Mooij [:jandem]]

Attachment: Bug 1604467 - Replace SRC_TRY source note with a JSOP_TRY bytecode operand. r?tcampbell!

JIT compilation and bytecode analysis now no longer depend on any source notes.

The deleted code to print the offset-to-catch in Disassemble1 goes back to
bug 680228, before the newer code coverage implementation. It's unlikely that
anyone is still depending on this and it's also confusing to print an offset
in the bytecode disassembly that's not actually an operand.

Comment 2

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/a590ca97f139
Replace SRC_TRY source note with a JSOP_TRY bytecode operand. r=tcampbell

Comment 3

[Natalia Csoregi [:nataliaCs]]

https://hg.mozilla.org/mozilla-central/rev/a590ca97f139