Created attachment 637953 [details] [diff] [review] patch ScopeCoordinate::slot isn't really the scope object slot, it's some sort of abstract index. It should be the slot (esp. with bug 767013). This patch also simplifies the conversions between frame-relative indices and scope-object-relative indices. The comment in jsscript.h should explain. Flagging dvander for review since this should require some minor tweaks in IM when this merges.
Thanks for the quick review! https://hg.mozilla.org/integration/mozilla-inbound/rev/e8bab55ac425
Backed out in https://hg.mozilla.org/integration/mozilla-inbound/rev/df25da024956 - it was a little bit short of actually compiling.
s/compiling/compiling for 32-bit/
Arg; the cursed JS_STATIC_ASSERT(sizeof(JSScript) % sizeof(Value) == 0). https://hg.mozilla.org/integration/mozilla-inbound/rev/0d9f7fb55226
Now the no-methodjit shell builds are burning (#winning): https://tbpl.mozilla.org/?tree=Mozilla-Inbound&noignore=1&jobname=spidermonkey