RFR 8136544: Call site switching to megamorphic causes incorrect property read

[Attila Szegedi]

+1. I’m happy with the very quick turnaround on fixing this.

> On Sep 16, 2015, at 12:46 PM, Sundararajan Athijegannathan <sundararajan.athijegannathan at oracle.com> wrote:
> 
> Hi,
> 
> * Changed NativeJavaImporter param name.
> * Commented on magic upper limit in ScopeTest.
> * Also added a .js test that explicitly sets the dynalink threshold and uses a limit slightly beyond that
> 
> http://cr.openjdk.java.net/~sundar/8136544/webrev.01/
> 
> Thanks
> -Sundar
> 
> On 9/16/2015 2:05 PM, Michael Haupt wrote:
>> Hi Sundar,
>> 
>> lower-case thumbs up, with two remarks:
>> * the new boolean argument in NativeJavaImporter should be called isScope, not isStrict
>> * ScopeTest could better illustrate the conditions it tests if it used a symbolic constant instead of 16 ("magic number") - can the megamorphism threshold be queried from the engine?
>> 
>> Best,
>> 
>> Michael
>> 
>>> Am 16.09.2015 um 06:27 schrieb Sundararajan Athijegannathan <sundararajan.athijegannathan at oracle.com <mailto:sundararajan.athijegannathan at oracle.com>>:
>>> 
>>> Please review http://cr.openjdk.java.net/~sundar/8136544/ <http://cr.openjdk.java.net/%7Esundar/8136544/> for https://bugs.openjdk.java.net/browse/JDK-8136544
>>> 
>>> Thanks,
>>> -Sundar
>> 
>> -- 
>> 
>> Oracle <http://www.oracle.com/>
>> Dr. Michael Haupt | Principal Member of Technical Staff
>> Phone: +49 331 200 7277 | Fax: +49 331 200 7561
>> OracleJava Platform Group | LangTools Team | Nashorn
>> Oracle Deutschland B.V. & Co. KG, Schiffbauergasse 14 | 14467 Potsdam, Germany
>> Green Oracle <http://www.oracle.com/commitment> 	Oracle is committed to developing practices and products that help protect the environment
>> 
>> 
>