Please review JDK-8061391

Hannes Wallnoefer hannes.wallnoefer at oracle.com
Thu Oct 23 11:57:07 UTC 2014


DeletedRangeArrayFilter.isEmpty should not override ArrayData.isEmpty as 
semantics are different. The easiest way to solve this would be to move 
new isEmpty method from ArrayData to ContinuousArrayData.

Otherwise +1, nice work!

Hannes

Am 2014-10-22 um 17:34 schrieb Marcus Lagergren:
> Please review this
>
> I added specialized builtins for concat, which are up to 10 times faster in microbenchmarks with just one element (or array) being concatenated.
> While doing this I happened upon some bugs with preventExtensions for arrays, and removed the hateful unreliable SwitchPoint from ContinuousArray, replacing non extensible arrays with an ArrayFilter, which is the way I should have done it in the first place.
> I also had to turn NoTypedArrayData into a structure that inherited ContinuousArray, or I threw away a lot of optimistic link sites to untouched arrays in ScriptObjects. This has other performance implications (good ones).
>
> ant test runs cleanly
> ant test262parallel runs cleanly
> octane runs with no regressions, some benchmarks like raytrace and crypto are even a bit faster.
>
> There are regressions in octane since the last time I checked, but not due to my fix. I will investigate and file bugs.
>
> Bug: https://bugs.openjdk.java.net/browse/JDK-8061391
> Webrev: http://cr.openjdk.java.net/~lagergren/8061391/
>
> Regards
> Marcus



More information about the nashorn-dev mailing list