Integrated: 8284760: Correct type/array element offset in LibraryCallKit::get_state_from_digest_object()

Roman Kennke rkennke at openjdk.java.net
Thu Apr 14 09:35:20 UTC 2022


On Tue, 12 Apr 2022 15:56:15 GMT, Roman Kennke <rkennke at openjdk.org> wrote:

> In LibraryCallKit::get_state_from_digest_object() we call array_element_address() with T_INT, even though the input array might also be T_BYTE or T_LONG. This doesn't currently matter much: array elements always start at the same offset regardless of the element type. In Lilliput I'm trying to tighten the start of array elements though, and this causes problems because I can do smaller alignments for T_BYTE and T_INT, but not for T_LONG.
> 
> See also: https://github.com/openjdk/lilliput/pull/41
> 
> Let's just use the correct type in array_element_address().
> 
> Testing:
>  - [x] tier1
>  - [x] jdk_security (includes relevant cipher tests)

This pull request has now been integrated.

Changeset: 2ba5cc41
Author:    Roman Kennke <rkennke at openjdk.org>
URL:       https://git.openjdk.java.net/jdk/commit/2ba5cc4163ccd944e2df917e5d617a78fa4ee75b
Stats:     22 lines in 2 files changed: 7 ins; 0 del; 15 mod

8284760: Correct type/array element offset in LibraryCallKit::get_state_from_digest_object()

Reviewed-by: roland, kvn

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

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


More information about the hotspot-compiler-dev mailing list