[master] RFR: 8300527: [Lilliput] Simplify and optimize loading of Klass* [v2]

[Aleksey Shipilev]

On Tue, 24 Jan 2023 17:42:11 GMT, Roman Kennke <rkennke at openjdk.org> wrote:

>> src/hotspot/share/runtime/synchronizer.cpp line 798:
>> 
>>> 796: static SharedGlobals GVars;
>>> 797: 
>>> 798: static markWord read_stable_mark(oop obj) {
>> 
>> Do you still need this method, or? If you still need it, let's not put it outside the `ObjectSynchronizer` "namespace".
>
> Originally, read_stable_mark() is static in upstream. A while back I put it into the ObjectSynchronizer namespace to reuse it in Shenandoah, but that is no longer needed, so I reverted the scope back to compilation-unit-private. Yes, we will still need it, when we make Lilliput switchable, but I'd keep it compilation-unit-private for the reasons mentioned.

All right then!

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

PR: https://git.openjdk.org/lilliput/pull/66