[master] RFR: OMWorld: Removal of flags, general cleanups

Coleen Phillimore coleenp at openjdk.org
Thu Jun 20 12:33:27 UTC 2024


On Thu, 20 Jun 2024 11:58:16 GMT, Axel Boldt-Christmas <aboldtch at openjdk.org> wrote:

> Cleanups in preparation for opening up a PR to mainline for [JDK-8315884](https://bugs.openjdk.org/browse/JDK-8315884).
> 
> @coleenp I incorporated #182 into this.

The cleanups look really good.  Can you fix the spelling of consistency?

src/hotspot/cpu/x86/sharedRuntime_x86.cpp line 69:

> 67:       __ testptr(result, markWord::monitor_value);
> 68:       __ jcc(Assembler::notZero, slowCase);
> 69:     }

Did I miss this one?

src/hotspot/share/runtime/globals.hpp line 1980:

> 1978:           "monitors rather than the first word of the object.")             \
> 1979:                                                                             \
> 1980:   product(int, LightweightFastLockingSpins, 13, DIAGNOSTIC,                 \

Good.  I like the new name.

src/hotspot/share/runtime/objectMonitor.cpp line 299:

> 297: }
> 298: 
> 299: #define assert_mark_word_concistency()                                         \

While you're here. The spelling is consistency.

src/hotspot/share/utilities/vmError.cpp line 595:

> 593:                  LockingMode == LM_MONITOR ? ", lm_monitors" :
> 594:                  LockingMode == LM_LEGACY ? ", lm_legacy" :
> 595:                  LockingMode == LM_LIGHTWEIGHT ? ", lm_lightweight" : "",

Without this, I guess the only way to know which locking mode is on is via. the command line in the hs_err file.  It didn't really belong here, but maybe we want to find somewhere else for it.  Not with this change.  It's good to clean this up for this change.

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

Marked as reviewed by coleenp (Committer).

PR Review: https://git.openjdk.org/lilliput/pull/183#pullrequestreview-2130286349
PR Review Comment: https://git.openjdk.org/lilliput/pull/183#discussion_r1647498292
PR Review Comment: https://git.openjdk.org/lilliput/pull/183#discussion_r1647497161
PR Review Comment: https://git.openjdk.org/lilliput/pull/183#discussion_r1647495754
PR Review Comment: https://git.openjdk.org/lilliput/pull/183#discussion_r1647494230


More information about the lilliput-dev mailing list