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

[Amit Kumar]

On Fri, 28 Jun 2024 13:08:57 GMT, Martin Doerr <mdoerr at openjdk.org> wrote:

>> What does "in future" mean? In the next couple of nanoseconds after this instruction executes? When someone changes this code?
>> You need to say, precisely, what this routine provides. That is to say, what is its postcondition. If some logic is not necessary for that postcondition, don't do it.
>> This routine sets result, or it sets flags, or both. Say which of these it is, the make sure that all code paths do so.
>
> I think the `z_cghi` is only used for the `asm_assert`. I'd guard both with `ifdef ASSERT`. Or combine it with the assert block below.

Done;

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

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