1119230 - Remove lingering PJS JIT bits

Status: RESOLVED FIXED

(Core :: JavaScript Engine: JIT, defect)

Description

[Lars T Hansen [:lth]]

The initial jit removal left behind some bits here and there, including in the public API, that are now (after other cleanups in the context code and GC) easy to remove.

Comment 1

[Lars T Hansen [:lth]]

Attachment: Remove sundry JIT bits

Comment 2

[Lars T Hansen [:lth]]

Attachment: Remove yet more JIT bits

The previous patch may have been premature, because here are yet more removals.  Feel free to r- these if you want them merged or if you have another scheme in mind than peacemeal hacking away at things until they're all gone.  (This one #if 0's most of ForkJoin.cpp.)

(Like the previous patch this one contains a point fix for the GC code.)

Comment 3

[Shu-yu Guo [:shu]]

Comment on attachment 8545927 [details] [diff] [review]
Remove sundry JIT bits

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

LGTM. Could you exclude the VMFunction changes? I will remove that more completely and simplify all the template hoohah in that file as well.

Comment 4

[Shu-yu Guo [:shu]]

Comment on attachment 8545973 [details] [diff] [review]
Remove yet more JIT bits

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

::: js/src/jsinfer.h
@@ +119,5 @@
>  
>  /*
>   * Execution Mode Overview
>   *
> + * JavaScript code executes sequentially.  Functions which have similar but not identical

Nit: Could you M-q the entire paragraph?

Comment 5

[Lars T Hansen [:lth]]

Try: https://treeherder.mozilla.org/ui/#/jobs?repo=try&revision=14e3a123fc5d

Comment 6

[Lars T Hansen [:lth]]

Try and try again: https://treeherder.mozilla.org/ui/#/jobs?repo=try&revision=756a70c50990

Comment 7

[Lars T Hansen [:lth]]

https://hg.mozilla.org/integration/mozilla-inbound/rev/2ff00342d088
https://hg.mozilla.org/integration/mozilla-inbound/rev/d43317c1d73f

Comment 8

[Carsten Book [:Tomcat]]

https://hg.mozilla.org/mozilla-central/rev/2ff00342d088
https://hg.mozilla.org/mozilla-central/rev/d43317c1d73f