Review request for JDK-8026016: too many relinks dominate avatar.js http benchmark

A. Sundararajan sundararajan.athijegannathan at oracle.com
Mon Oct 14 02:31:30 PDT 2013


Looks good to me

-Sundar

On Monday 14 October 2013 01:46 PM, Hannes Wallnoefer wrote:
> Please review JDK-8026016: too many relinks dominate avatar.js http 
> benchmark.
>
> http://cr.openjdk.java.net/~hannesw/8026016/webrev.02/
>
> Apart from the fix to let callsites for non-existant properties go 
> megamorphic (while preserving noSuchMethod functionality) this also 
> removes some minor inefficiencies in ScriptObject. set() methods used 
> to check key for array index twice, and has*() methods unnecessarily 
> created an FindProperty object.
>
> Thanks,
> Hannes



More information about the nashorn-dev mailing list