RFR: 8231269: CompileTask::is_unloaded is slow due to JNIHandles type checks [v19]

[Kim Barrett]

On Mon, 26 May 2025 18:58:42 GMT, Aleksey Shipilev <shade at openjdk.org> wrote:

> > Not sure what our opinion is w.r.t. `mutable`, but how do we feel about typing the spin lock as `mutable` and keep `is_safe()` and `method*()` const.
> 
> I like this a lot! Dropping `const` just to satisfy spin lock (an implementation detail) felt really awkward. New version uses `mutable`.

Just a drive-by reply.  `mutable` is a C++98 (and before, I think) feature, with many uses in HotSpot.  Using it here
seems fine to me.

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

PR Comment: https://git.openjdk.org/jdk/pull/24018#issuecomment-3053728810