[lworld] RFR: 8357785: [lworld] TestResolvedJavaType fails due to unexpected getInstanceFields order [v3]

Marc Chevalier mchevalier at openjdk.org
Fri Jul 25 09:40:54 UTC 2025


On Fri, 25 Jul 2025 08:21:24 GMT, Tobias Hartmann <thartmann at openjdk.org> wrote:

>> Marc Chevalier has updated the pull request incrementally with one additional commit since the last revision:
>> 
>>   Second round of comments
>
> src/hotspot/share/code/nmethod.cpp line 3846:
> 
>> 3844:       if (!did_name)
>> 3845:         stream->print("%s", type2name(t));
>> 3846:       // If the entry has a non-default sort_offset, it must be a null marker
> 
> Comment needs to be updated.

I think it can just be removed: if it's a null marker, we print that it's a null marker. I think we don't need it.

> src/hotspot/share/runtime/sharedRuntime.cpp line 2869:
> 
>> 2867:             if (bt == T_OBJECT) {
>> 2868:               // Nullable inline type argument, insert InlineTypeNode::NullMarker field right after T_METADATA delimiter
>> 2869:               // Set the sort_offset so that the field is detected as null marker by nmethod::print_nmethod_labels.
> 
> This comment needs to be updated. It still mentions the `sort_offset`.

I think we can do without the second line. The first says we include a null marker, and the second gives true to the null_marker parameter. Pretty straightforward.

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

PR Review Comment: https://git.openjdk.org/valhalla/pull/1511#discussion_r2230642917
PR Review Comment: https://git.openjdk.org/valhalla/pull/1511#discussion_r2230645885


More information about the valhalla-dev mailing list