RFR: 8331126: [s390x] secondary_super_cache does not scale well [v4]

[Martin Doerr]

On Mon, 24 Jun 2024 14:02:15 GMT, Lutz Schmidt <lucy at openjdk.org> wrote:

>> Amit Kumar has updated the pull request incrementally with two additional commits since the last revision:
>> 
>>  - Update src/hotspot/cpu/s390/macroAssembler_s390.cpp
>>  - rename: r_scratch to r_result in repne_scan method
>
> src/hotspot/cpu/s390/macroAssembler_s390.cpp line 3275:
> 
>> 3273:   call_stub(StubRoutines::lookup_secondary_supers_table_slow_path_stub());
>> 3274: 
>> 3275:   z_bru(L_done); // pass whatever result we got from a slow path
> 
> This one branch could be saved by using "load immediate on condition". But it's after slow path processing.

Right, looks like we only reach here with "false" condition or after return from the stub which should have set the condition code accordingly, too (should be checked / enforced!).

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

PR Review Comment: https://git.openjdk.org/jdk/pull/19544#discussion_r1655423255