RFR: 8166902: Nested object literal property maps not reset in optimistic recompilation

[Vivin Suresh Paliath]

Thanks for resolving this so quickly!

On Fri, Sep 30, 2016 at 10:30 AM, Hannes Wallnöfer <
hannes.wallnoefer at oracle.com> wrote:

> Thanks to both of you!
>
> Attila: the TypeEvaluator (which is the only thing that will cause such as
> map change AFAICT) was added after the continuation code in question, so I
> think it was actually correct back then ;-)
>
> Hannes
>
>
> > Am 30.09.2016 um 18:29 schrieb Attila Szegedi <szegedia at gmail.com>:
> >
> > +1; good work. I'm kinda embarrassed I didn’t think of this case…
> >
> > Attila.
> >
> >> On 30 Sep 2016, at 18:18, Marcus Lagergren <marcus at lagergren.net>
> wrote:
> >>
> >> +1. Good catch, Hannes!
> >>
> >> /M
> >>
> >>> On 30 Sep 2016, at 16:46, Hannes Wallnöfer <
> hannes.wallnoefer at oracle.com> wrote:
> >>>
> >>> Please review 8166902: Nested object literal property maps not reset
> in optimistic recompilation:
> >>>
> >>> Bug: https://bugs.openjdk.java.net/browse/JDK-8166902
> >>> Webrev: http://cr.openjdk.java.net/~hannesw/8166902/webrev.00/
> >>>
> >>> The original assumption was that the property map only has to be
> replaced in the top level object literal when optimistic recompilation
> happens within a nested object literal. However, recompilation may also
> create different maps on parent object literals as more information about
> types may be available.
> >>>
> >>> Thanks,
> >>> Hannes
> >>
> >
>
>


-- 
*[vivin.net <http://vivin.net> :: github <http://github.com/vivin> ::
linkedin <https://www.linkedin.com/in/vivin>]*