RFR: 8276921: G1: Remove redundant failed evacuation regions calculation in RemoveSelfForwardPtrHRClosure

Albert Mingkun Yang ayang at openjdk.java.net
Fri Nov 12 11:00:35 UTC 2021


On Fri, 12 Nov 2021 02:00:57 GMT, Hamlin Li <mli at openjdk.org> wrote:

> This is a minor optimization.
> We already have the number of failed evacuation region numbers, there is no need to calculate it on fly, at least in product code, so make it debug only, and add assert to verify the result.

I think `_num_failed_regions` can be dropped completely; it's used for verification only. `~RemoveSelfForwardPtrsTask` can go as well as a cascading effect.

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

PR: https://git.openjdk.java.net/jdk/pull/6362


More information about the hotspot-gc-dev mailing list