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