RFR: 8373295: Wrong log tag for UseCompressedOops ergo setting
Ioi Lam
iklam at openjdk.org
Tue Dec 9 16:13:01 UTC 2025
On Tue, 9 Dec 2025 05:10:55 GMT, Yasumasa Suenaga <ysuenaga at openjdk.org> wrote:
>> Please review this trivial change: Ergonomic settings of `UseCompressedOops` should be logged with `log_debug(gc, heap, coops)`
>>
>>
>> Note: this is the same log channel used just a few lines above.
>>
>> https://github.com/openjdk/jdk/blob/c9ab330b7bdd3cc2410ffdb336a63aa0ac7256a3/src/hotspot/share/runtime/arguments.cpp#L1579-L1586
>
> I completely agree with your change!
>
> But I received review that it should not be changed from CDS tag (I know it is `aot` now). See review thread in my past proposal for details: https://mail.openjdk.org/pipermail/hotspot-runtime-dev/2019-October/036315.html
>
> I'm happy to approve if we can accept.
Thanks @YaSuenag @dholmes-ora for the review
-------------
PR Comment: https://git.openjdk.org/jdk/pull/28716#issuecomment-3633064865
More information about the hotspot-runtime-dev
mailing list