Review request for JDK-8006570

Jim Laskey (Oracle) james.laskey at oracle.com
Tue Jan 22 04:46:07 PST 2013


Looks good.

On 2013-01-22, at 7:01 AM, Hannes Wallnoefer <hannes.wallnoefer at oracle.com> wrote:

> I've uploaded a new webrev based mostly on feedback by Sundar.
> 
> http://cr.openjdk.java.net/~hannesw/8006570/
> 
> Changes to the first webrev:
> 
> - Test now also covers this-conversion cases that were not subject of this bug.
> 
> - NashornGuardedInvocation does no longer make use of the NashornCallSiteDescriptor.CALLSITE_STRICT flag. The purpose of NashornGuardedInvocation never was to track strictness of the callsite, but of the invocation target function (Javadoc comments were misleading and have also been fixed).
> 
> What we really need to know from NashornGuardedInvocation is whether the target function is a non-strict script function (i.e. neither built-in nor strict) so I changed the NashornGuardedInvocation.isStrict() methods to NashornGuardedInvocation.isNonStrict() with these semantics (neither strict nor built-in).
> 
> Most checks for target function strictness already did this correctly (i.e. check for both not-strict and not-builtin) but the one in PrimitiveLookup.lookupPrimitive() did not, making string method invocation 2-3 times more expensive because of the unnecessary string wrapper creation (I'll add a String microbenchmark later as part of JDK-8006408).
> 
> Hannes
> 
> Am 2013-01-21 17:28, schrieb Hannes Wallnoefer:
>> Please review JDK-8006570 at http://cr.openjdk.java.net/~hannesw/8006570/.
>> 
>> Thanks,
>> Hannes
> 



More information about the nashorn-dev mailing list