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

[Vladimir Ivanov]

On Tue, 15 Oct 2024 10:29:01 GMT, Andrew Haley <aph at openjdk.org> wrote:

>> 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.

I agree that a full retest would be an overkill. I submitted limited testing with the latest patch and results look good.

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

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