On Wed, 8 Apr 2026 12:31:03 GMT, Stefan Karlsson <[email protected]> wrote:
> @caspernorrbin Started cleaning up markWord.hpp with > https://github.com/openjdk/valhalla/pull/2238, but while reviewing that patch > I realized that it would be good to make a number of simplifications before > proceeding. I talked that through with Casper and @Arraying. Those changes > have now been done, and I've taken a pass through markWord.hpp to updated the > code in the spirit of what Casper did and some of the things we three talked > about. > > I've also made updates and clarifications to the comments. > > One main thing about this proposal is that I want it to be crystal clear when > we talk about bit and when we talk about a mask. To make sure that the bit > constants only ever refers to one bit I've added helpers to verify that this > is the case. This pull request has now been integrated. Changeset: 8d83bd9f Author: Stefan Karlsson <[email protected]> URL: https://git.openjdk.org/valhalla/commit/8d83bd9f7a63795a6a158ce6e3f0faa459b6a8a2 Stats: 83 lines in 4 files changed: 26 ins; 14 del; 43 mod 8374147: [lworld] Cleanup markword bit masks and constants Co-authored-by: Casper Norrbin <[email protected]> Reviewed-by: cnorrbin, phubner ------------- PR: https://git.openjdk.org/valhalla/pull/2310
