Integrated: 8374518: C1: Remove dead LinearScanStatistic::Counter::counter_fpu_stack
Christian Hagedorn
chagedorn at openjdk.org
Tue Jan 6 10:27:30 UTC 2026
On Mon, 5 Jan 2026 13:28:41 GMT, Christian Hagedorn <chagedorn at openjdk.org> wrote:
> The usages of `LinearScanStatistic::Counter::counter_fpu_stack` were removed with [JDK-8351156](https://bugs.openjdk.org/browse/JDK-8351156) when the FPU stack support was removed. But the definition in the `Counter` enum was missed to clean up. When now printing the counters with `-XX:+CountLinearScan -XX:+CITime`, we reach a `ShouldNotReach` in `LinearScanStatistic::counter_name()` for `Counter::counter_fpu_stack` because there is no longer an entry there - it's dead.
>
> I added a simple hello world sanity test that runs with `-XX:+CountLinearScan -XX:+CITime`.
>
> Thanks,
> Christian
This pull request has now been integrated.
Changeset: 938bbd5b
Author: Christian Hagedorn <chagedorn at openjdk.org>
URL: https://git.openjdk.org/jdk/commit/938bbd5b604e990514b64a0451ed1bceb07eb23b
Stats: 39 lines in 2 files changed: 37 ins; 1 del; 1 mod
8374518: C1: Remove dead LinearScanStatistic::Counter::counter_fpu_stack
Reviewed-by: thartmann, mdoerr
-------------
PR: https://git.openjdk.org/jdk/pull/29037
More information about the hotspot-compiler-dev
mailing list