881410 - Remove some TI code that was only used by JM

Status: RESOLVED FIXED

(Core :: JavaScript Engine, defect)

Description

[Jan de Mooij [:jandem]]

Attachment: Patch

Various unused fields like chunkIndex, and recompilation no longer needs the script + pc, just the script.

Comment 1

[Brian Hackett [Laid off!]]

Comment on attachment 760533 [details] [diff] [review]
Patch

Review of attachment 760533 [details] [diff] [review]:
-----------------------------------------------------------------

::: js/src/jsinfer.cpp
@@ +2364,5 @@
>       * Trigger recompilation of the script itself, if code was not previously
>       * compiled with the specified information.
>       */
> +    if (!JITCodeHasCheck(script, kind))
> +        cx->compartment->types.addPendingRecompile(cx, script);

JITCodeHasCheck will only return true when the script has no Ion jitcode at all, in which case addPendingRecompile shouldn't do anything.  So I think that JITCodeHasCheck can additionally be removed, along with the RecompileKind enum.

Comment 2

[Jan de Mooij [:jandem]]

https://hg.mozilla.org/integration/mozilla-inbound/rev/407dd32db3b0

Comment 3

[Ryan VanderMeulen [:RyanVM]]

https://hg.mozilla.org/mozilla-central/rev/407dd32db3b0