Created attachment 637953 [details] [diff] [review]
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!
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).
Now the no-methodjit shell builds are burning (#winning):