RFR: 8359394: Shenandoah: GC cause cleanup
William Kemper
wkemper at openjdk.org
Fri Jun 13 00:15:29 UTC 2025
On Thu, 12 Jun 2025 23:42:41 GMT, William Kemper <wkemper at openjdk.org> wrote:
> We can remove `GCCause::_no_cause_specified` and need to add a case for `_shenandoah_humongous_allocation_failure` in `GCCause::to_string`. We are also missing `_shenandoah_humongous_allocation_failure` in the `SA` analog.
src/hotspot/share/gc/shared/gcCause.cpp line 92:
> 90: return "Diagnostic Command";
> 91:
> 92: case _shenandoah_stop_vm:
Just reordered this case to match enum's declaration order.
src/hotspot/share/gc/shared/gcCause.hpp line 118:
> 116: inline static bool is_allocation_failure_gc(GCCause::Cause cause) {
> 117: // _allocation_failure is the generic cause a collection for allocation failure
> 118: return cause == GCCause::_allocation_failure;
Shenandoah does not use this method. Nobody else needs to know about this Shenandoah specific value.
-------------
PR Review Comment: https://git.openjdk.org/jdk/pull/25789#discussion_r2143900441
PR Review Comment: https://git.openjdk.org/jdk/pull/25789#discussion_r2143904488
More information about the shenandoah-dev
mailing list