RFR: 8278548: G1: Remove unnecessary check in forward_to_block_containing_addr

Hamlin Li mli at openjdk.java.net
Fri Dec 10 12:33:13 UTC 2021


On Fri, 10 Dec 2021 11:36:09 GMT, Albert Mingkun Yang <ayang at openjdk.org> wrote:

> Simple change of removing an always-false `if`.
> 
> Test: I added `assert(false)` inside the `if`, and no failure in tier1-6.

Not sure if I'm overreact, but IMHO it might be more safe to replace it with an assert. How do you think about it?

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

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



More information about the hotspot-gc-dev mailing list