RFR 8200215: 17th loop of "let foo = ''"; throws ReferenceError

[Sundararajan Athijegannathan]

Hi Attila,

Yes, new Map has to be created with a property with modified flags. And 
yes, 17 was the clue :)

Thanks,
-Sundar

On 26/03/18, 5:48 PM, Attila Szegedi wrote:
> +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