723767 - IonMonkey: Fix nomethodjit build

Status: RESOLVED FIXED

(Core :: JavaScript Engine, defect)

Description

[David Anderson [:dvander] - inactive, e-mail if emergency]

Attachment: fix

Getting tbpl green.

Comment 1

[Chris Leary [:cdleary] (not checking bugmail)]

Comment on attachment 594020 [details] [diff] [review]
fix

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

::: js/src/jsinfer.cpp
@@ +2229,3 @@
>  #if defined(JS_METHODJIT)
>      mjit::JITScript *jit = info.script->getJIT(info.constructing);
> +    bool hasJITCode = jit && jit->chunkDescriptor(info.chunkIndex).chunk;

Hmm... is there a good place to document that defined(JS_ION) implies defined(JS_METHODJIT)? Maybe it doesn't matter.

Comment 2

[David Anderson [:dvander] - inactive, e-mail if emergency]

Probably not... if we ever get rid of JM, JS_ION should become JS_METHODJIT. IonMonkey depends on assembler/* too which is only built if JM is.

http://hg.mozilla.org/projects/ionmonkey/rev/3b67e0b8192d