RFR: 8366498: Simplify ClassFileParser::parse_super_class [v2]

Ioi Lam iklam at openjdk.org
Mon Sep 1 08:03:41 UTC 2025


On Mon, 1 Sep 2025 07:54:57 GMT, David Holmes <dholmes at openjdk.org> wrote:

>> I think it help people who wonder why `_super_klass` is not updated in this case. Maybe I can change the assignment to
>> 
>> 
>> asssert(_super_klass == nullptr, "already initialized");
>
> I would just have:
> 
> else {
>   // The class is Object - so no superclass
> }

My asserts say the same thing, except they comes with proofs that they are not lying.

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

PR Review Comment: https://git.openjdk.org/jdk/pull/27026#discussion_r2313189601


More information about the hotspot-runtime-dev mailing list