Uses CallInfo when inlining natives -- part of a cleanup effort that's a prerequisite for Bug 837312. There are some subtleties: most importantly, argv[n] is now callInfo.getArg(n-1), with explicit functions added for querying |this| to match the CallInfo interface. Passes jit-tests.

Comment on attachment 710517 [details] [diff] [review] patch Review of attachment 710517 [details] [diff] [review]: ----------------------------------------------------------------- Cool, thanks for doing this! FYI getInlineThisTypeSet/getInlineArgTypeSet will go into CallInfo as of bug 796114 ::: js/src/ion/MCallOptimize.cpp @@ +540,5 @@ > if (getInlineReturnType() != MIRType_Double) > return InliningStatus_NotInlined; > > + MIRType arg0Type = getInlineArgType(callInfo, 0); > + MIRType arg1Type = getInlineArgType(callInfo, 1); Because you name the args a little bit later, I would name their type with ***Type s/arg0Type/baseType s/arg1Type/exponentType @@ +550,5 @@ > > + callInfo.unwrapArgs(); > + > + MDefinition *input = callInfo.getArg(0); > + MDefinition *power = callInfo.getArg(1); I think the names for the inputs to a power is base and exponent s/input/base s/power/exponent