RFR: 8263125: During deoptimization vectors should reassign scalarized payload after all objects are reallocated. [v2]

[Vladimir Kozlov]

On Fri, 12 Mar 2021 08:48:50 GMT, Richard Reingruber <rrich at openjdk.org> wrote:

>> Vladimir Kozlov has updated the pull request incrementally with one additional commit since the last revision:
>> 
>>   Update Copyright year
>
> Hello Vladimir,
> 
> your change looks good to me. You might want to add the refactoring Vladimir
> suggested.
> 
> May I ask why there is a special case to reallocate a vectors payload at all. In
> other words: why is the method VectorSupport::allocate_vector_payload_helper()
> needed? Is it for support of VectorMask?
> 
> Thanks, Richard.

> > I renamed incorrect eliminate_* names for methods which restore/reallocate objects and locks
> 
> Fully agree that `eliminate_allocations`/`eliminate_locks` are misleading, but `restore_*` still look a bit confusing to me.
> What do you think about `rematerialize_objects`/`rematerialize_scalarized_objects`/`relock_objects`/`restore_eliminated_locks`?

I selected `rematerialize_objects` and `restore_eliminated_locks`.

-------------

PR: https://git.openjdk.java.net/jdk/pull/2924