[master] RFR: 8317812: [Lilliput] Make C2 LoadNKlassCompactHeader more robust [v8]

[Aleksey Shipilev]

On Thu, 12 Oct 2023 09:03:56 GMT, Roman Kennke <rkennke at openjdk.org> wrote:

>> Lilliput's C2 code for generating LoadNKlass currently assumes that the disp of the incoming address is klass_offset_in_bytes. It then extracts the base register and loads from the mark_offset_in_bytes instead.
>> Sometimes (apparently very rarely) it happens that C2 emits code that pre-adds obj+klass_offset into a register, and uses that as base, but with offset 0. In this case we would trip the assert or crash in release builds.
>> 
>> - [x] specjvm derby (which triggered the bug)
>> - [x] tier1 x86_64 +UCOH
>> - [x] tier1 aarch64 +UCOH
>
> Roman Kennke has updated the pull request incrementally with one additional commit since the last revision:
> 
>   Cleanup

Marked as reviewed by shade (Committer).

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

PR Review: https://git.openjdk.org/lilliput/pull/111#pullrequestreview-1673665559