Closed Bug 623277 Opened 9 years ago Closed 9 years ago
Failed to build with --disable-methodjit
/home/adrake/src/tm/js/src/jsinterpinlines.h: In member function ‘bool js::InvokeSessionGuard::invoke(JSContext*) const’: /home/adrake/src/tm/js/src/jsinterpinlines.h:589:43: error: ‘struct JSScript’ has no member named ‘getJIT’ Patch forthcoming.
Adds a ifdef JS_METHODJIT around the condition involving getJIT.
shell/js.cpp needs a similar patch, so here is a unified one. David, would you be willing to review this, or tell me who would?
Attachment #502317 - Flags: review?(dmandelin)
Attachment #502317 - Flags: review?(dmandelin) → review+
I'm going to use this bug for all build failures with --disable-methodjit, since we have 3 of them right now that I know of and various parts of them have already been fixed. The patch in this bug is nearly all that's left, although part of another of adrake's patches had a warning fix, and the patch attached to this bug has Windows line endings mixed in, so I'll attach a new one and dupe the other bugs here.
Summary: Failed to build jsapi.cpp with --disable-methodjit → Failed to build with --disable-methodjit
Darn. No simple way to carry over r=dmandelin?
Authorship got mixed up, but: http://hg.mozilla.org/tracemonkey/rev/7372dac81910
cdleary-bot mozilla-central merge info: http://hg.mozilla.org/mozilla-central/rev/7372dac81910
Status: NEW → RESOLVED
Closed: 9 years ago
Resolution: --- → FIXED
You need to log in before you can comment on or make changes to this bug.