[lworld] RFR: 8373494: [lworld] markWord preservation for ParallelGC [v4]

Joel Sikström jsikstro at openjdk.org
Mon Dec 15 10:04:47 UTC 2025


On Fri, 12 Dec 2025 19:25:14 GMT, Coleen Phillimore <coleenp at openjdk.org> wrote:

> EnableValhalla is a property that determines the answer to safe_to_read_header(), so I think it should go there.

I don't see EnableValhalla as something that determines the answer to safe_to_read_header(). Rather, we only need to check safe_to_read_header() if Valhalla is enabled. I think it's clearer to show at the call site that this is a Valhalla-specific concern, since checking if the header is safe to read or not isn't relevant otherwise.

I'll go ahead and add a comment to the current version of safe_to_read_header().

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

PR Review Comment: https://git.openjdk.org/valhalla/pull/1785#discussion_r2618752202


More information about the valhalla-dev mailing list