RFR: 8331341: secondary_super_cache does not scale well: C1 and interpreter [v23]

Andrew Haley aph at openjdk.org
Tue Oct 15 10:31:16 UTC 2024


On Mon, 14 Oct 2024 19:18:56 GMT, Dean Long <dlong at openjdk.org> wrote:

>> I don't think we have a spare register,
>
> What if you changed lines 1820-1821 like so:
> 
> eor(temp2, slot, (u1)(Klass::SECONDARY_SUPERS_TABLE_SIZE - 1));
> lslv(temp2, r_bitmap, temp2);

Yes, you're right. Please look at it now.

@iwanowww , do you think this needs a full re-test? This is a local change, and I think it's self-contained.

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

PR Review Comment: https://git.openjdk.org/jdk/pull/19989#discussion_r1800887071


More information about the hotspot-dev mailing list