RFR: 8365378: Redundant code in Deoptimization::print_statistics

Coleen Phillimore coleenp at openjdk.org
Thu Aug 21 13:03:52 UTC 2025


On Tue, 12 Aug 2025 12:31:28 GMT, Paul Hübner <duke at openjdk.org> wrote:

> Hi all,
> 
> The `Deoptimization::print_statistics` function has a conditional that is never invoked. `bc_case` is bound in a for loop and will always be < `BC_CASE_LIMIT`. Therefore, the entire condition will never hold, and the branch never execute. This change removes the dead code in order to increase maintainability.
> 
> I've run hotspot:tier1 to hotspot:tier3, in which I observed no test failures.

Looks good!

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

Marked as reviewed by coleenp (Reviewer).

PR Review: https://git.openjdk.org/jdk/pull/26744#pullrequestreview-3140555973


More information about the hotspot-dev mailing list