RFR: 8346457: AOT cache creation crashes with "assert(pair_at(i).match() < pair_at(i+1).match()) failed: unsorted table entries"

[David Holmes]

On Mon, 23 Dec 2024 23:53:13 GMT, Calvin Cheung <ccheung at openjdk.org> wrote:

> Avoid calling `InstanceKlass::compute_has_loops_flag_for_methods()` when a class has failed verification during CDS dump. It is to avoid assert in subsequent byte code verification such as in `Bytecode_lookupswitch::verify()`.
> 
> Testing: tiers 1 - 4.

This seems quite reasonable.

Thanks

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

Marked as reviewed by dholmes (Reviewer).

PR Review: https://git.openjdk.org/jdk/pull/22871#pullrequestreview-2531401374