RFR 8200215: 17th loop of "let foo = ''"; throws ReferenceError
    Attila Szegedi 
    szegedia at gmail.com
       
    Mon Mar 26 12:18:30 UTC 2018
    
    
  
+1, looks good. Presumably this happened on reversal to megamorphic so setObject was linked and it didn’t properly handle this case… the crux of the matter, if I understand it correctly is that once the property is set, the NEEDS_DECLARATION flag needs to be removed, right?
I presume the fast-linked (non-megamorphic) paths handle this correctly already, otherwise we would’ve seen the bug on first iteration, not on 17th :-)
Attila.
> On 2018. Mar 26., at 14:00, Sundararajan Athijegannathan <sundararajan.athijegannathan at oracle.com> wrote:
> 
> Please review http://cr.openjdk.java.net/~sundar/8200215/webrev.00/
> Bug: https://bugs.openjdk.java.net/browse/JDK-8200215
> 
> Thanks,
> -Sundar
    
    
More information about the nashorn-dev
mailing list