Integrated: 8323595: is_aligned(p, alignof(OopT))) assertion fails in Jetty without compressed OOPs

Patricio Chilano Mateo pchilanomate at openjdk.org
Thu Jan 18 12:19:24 UTC 2024


On Wed, 17 Jan 2024 15:07:04 GMT, Patricio Chilano Mateo <pchilanomate at openjdk.org> wrote:

> This is an oversight in one of the asserts I added as part of the 8320275 fix. Variable `end` cannot be passed to bit_index_for() since it can be unaligned if UseCompressedOops is off. We actually only need `effective_end` in the assert since we only want to check that if `end` was unaligned there is indeed no bit set for that last stack slot of the argument area.
> I run the failing test several times with -XX:-UseCompressedOops. I also run the upper loom tiers with that flag since it seems it's not being tested as much. 
> 
> Thanks,
> Patricio

This pull request has now been integrated.

Changeset: 52f787f6
Author:    Patricio Chilano Mateo <pchilanomate at openjdk.org>
URL:       https://git.openjdk.org/jdk/commit/52f787f675146d98d3e2338b14b7cd6b1dba7bb8
Stats:     1 line in 1 file changed: 0 ins; 0 del; 1 mod

8323595: is_aligned(p, alignof(OopT))) assertion fails in Jetty without compressed OOPs

Reviewed-by: fparain, dlong

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

PR: https://git.openjdk.org/jdk/pull/17465


More information about the hotspot-runtime-dev mailing list