RFR: 8326612: Parallel: remove redundant assertion from ScavengeRootsTask

Thomas Schatzl tschatzl at openjdk.org
Thu Feb 29 08:28:10 UTC 2024


On Sat, 24 Feb 2024 09:29:20 GMT, Richard Reingruber <rrich at openjdk.org> wrote:

> This change removes a redundant assertion from `ScavengeRootsTask`.
> The assertion is redundant because `_old_gen` is initialized to `old_gen` at L296 which is dereferenced at L297 before the assertion is reached.

Since `old_gen` is already dereferenced in the initializer list, the code would already crash before reaching this assert anyway, so it is useless.

You do need to file a CR and fix the title though until this can be moved along.

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

Marked as reviewed by tschatzl (Reviewer).

PR Review: https://git.openjdk.org/jdk/pull/17994#pullrequestreview-1908220960
PR Comment: https://git.openjdk.org/jdk/pull/17994#issuecomment-1970620649


More information about the hotspot-gc-dev mailing list