RFR: Use method offset as the hash for storing nmethods

Vladimir Kozlov kvn at openjdk.org
Thu Aug 14 14:05:30 UTC 2025


On Thu, 14 Aug 2025 13:59:22 GMT, Vladimir Kozlov <kvn at openjdk.org> wrote:

>>> An other change could be preserving method's name in AOT cache only in debug VM for this compares. For printing we can get name from entry->method()->name_and_sig_as_C_string().
>> 
>> Right, that was in my list as well. I was thinking of not storing the name at all. But I guess we can keep it for debug builds.
>
>> > As you said we can't use pointers (base could be different).
>> > I suggest to keep original code here.
>> 
>> But we should have already restored entry->method() correctly based on current runtime's base address. So method() should always be same as entry->method().
> 
> Good point. I agree with this then. Please, print full name too in the assert.

May be print both names, saved and from `method()->name_and_sig_as_C_string()` only if they are different.

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

PR Review Comment: https://git.openjdk.org/leyden/pull/91#discussion_r2276734609


More information about the leyden-dev mailing list