[pr/110] RFR: 8317812: [Lilliput] Make C2 LoadNKlassCompactHeader more robust

Aleksey Shipilev shade at openjdk.org
Wed Oct 11 09:39:47 UTC 2023


On Wed, 11 Oct 2023 09:20:01 GMT, Roman Kennke <rkennke at openjdk.org> wrote:

>> In fact, I don't quite see why `x86_64.ad` cannot do what `aarch64.ad` does:
>> 
>> 
>> __ load_nklass_compact($dst$$Register, $mem$$base$$Register, $mem$$index$$Register, $mem$$scale, $mem$$disp);
>
> Problem is the index=4 does not decode to noreg (it decodes to rsp) and scale=0 does not decode to no_scale (it's invalid, I think). Hence the extra hoops there.

Ooof. Okay then.

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

PR Review Comment: https://git.openjdk.org/lilliput/pull/111#discussion_r1354576727


More information about the lilliput-dev mailing list