RFR: Use method offset as the hash for storing nmethods [v3]

[Vladimir Kozlov]

On Thu, 14 Aug 2025 21:42:46 GMT, Ashutosh Mehra <asmehra at openjdk.org> wrote:

>> Currently AOTCodeEntry for nmethod uses a hash obtained from the method holder's name, method name and signature.   This is not very reliable.  With https://github.com/openjdk/leyden/pull/90, AOTCodeEntry is always linked to its corresponding Method, so we should be able to use the Method pointer as the hash for AOTCodeEntries for nmethod. However, the problem with pointers are not stable across runs. So instead of the pointer, we can use the offset of the Method in AOTCache. This should work because the Method is guaranteed to be in AOTCache.
>> This PR updates the code to use Method offset as the hash for AOTCodeEntries for nmethods.
>
> Ashutosh Mehra has updated the pull request incrementally with two additional commits since the last revision:
> 
>  - Remove Method* from AOTCodeEntry
>    
>    Signed-off-by: Ashutosh Mehra <asmehra at redhat.com>
>  - Fix compile failure
>    
>    Signed-off-by: Ashutosh Mehra <asmehra at redhat.com>

This looks good. I will test it.

> 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.

Lets do it in separate changes, this is not urgent.

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

PR Comment: https://git.openjdk.org/leyden/pull/91#issuecomment-3190027688