Integrated: 8356647: C2: Excessively strict assert in PhaseIdealLoop::do_unroll
Marc Chevalier
mchevalier at openjdk.org
Wed May 28 13:29:01 UTC 2025
On Mon, 19 May 2025 06:43:38 GMT, Marc Chevalier <mchevalier at openjdk.org> wrote:
> This assert seems a bit too tight. See the JBS issue to check the math: the bound of `trip_count` should be `<= 2^31`, while the current bound is ` < (julong)max_juint/2` = floor((2^32-1)/2) = (2^32-2) / 2 = 2^31-1.
This pull request has now been integrated.
Changeset: 4b9290af
Author: Marc Chevalier <mchevalier at openjdk.org>
URL: https://git.openjdk.org/jdk/commit/4b9290af0a46bdf662735c24d00732a4c1601102
Stats: 59 lines in 3 files changed: 57 ins; 0 del; 2 mod
8356647: C2: Excessively strict assert in PhaseIdealLoop::do_unroll
Reviewed-by: chagedorn, epeter, dlong
-------------
PR: https://git.openjdk.org/jdk/pull/25295
More information about the hotspot-compiler-dev
mailing list