1132564 - Remove separate ExecutableAllocator for Ion code

Status: RESOLVED FIXED

(Core :: JavaScript Engine: JIT, defect)

Description

[Jan de Mooij [:jandem]]

Attachment: Part 1 - Remove Ion allocator

After bug 1091912, this is no longer necessary and we can use a single allocator for all JIT code.

Comment 1

[Jan de Mooij [:jandem]]

Attachment: Part 2 - Move ExecutableAllocator into JitRuntime

With YARR gone, we only use ExecutableAllocator for JitCode, and JitCodes are only allocated when we have a JitRuntime, so we can simply move ExecutableAllocator into JitRuntime.

This patch also has some minor cleanup.

Comment 2

[Luke Wagner [:luke]]

Comment on attachment 8563561 [details] [diff] [review]
Part 1 - Remove Ion allocator

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

My favorite kind of review.

Comment 3

[Luke Wagner [:luke]]

Comment on attachment 8563565 [details] [diff] [review]
Part 2 - Move ExecutableAllocator into JitRuntime

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

Nice.

Comment 4

[Jan de Mooij [:jandem]]

https://hg.mozilla.org/integration/mozilla-inbound/rev/7dc41de40625
https://hg.mozilla.org/integration/mozilla-inbound/rev/4d4b564c9d84

Comment 5

[Carsten Book [:Tomcat]]

https://hg.mozilla.org/mozilla-central/rev/7dc41de40625
https://hg.mozilla.org/mozilla-central/rev/4d4b564c9d84