RFR: 8349088: De-virtualize Codeblob and nmethod [v8]

Vladimir Kozlov kvn at openjdk.org
Fri Feb 14 23:14:23 UTC 2025


On Thu, 13 Feb 2025 19:27:19 GMT, Chris Plummer <cjplummer at openjdk.org> wrote:

>> Vladimir Kozlov has updated the pull request incrementally with one additional commit since the last revision:
>> 
>>   rename SA argument
>
> src/jdk.hotspot.agent/share/classes/sun/jvm/hotspot/code/CodeCache.java line 97:
> 
>> 95:   // cbAddr - address of a code blob
>> 96:   // cbPC   - address inside of a code blob
>> 97:   public CodeBlob createCodeBlobWrapper(Address cbAddr, Address cbPC) {
> 
> Can you change findBlobUnsafe() above also? That's where the naming problem originated.

After some thoughts I think `PC` is not usually used by us.
I renamed `cbAddr` to `cbStart` and `cbPC`/`start` to `addr` in this whole file.

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

PR Review Comment: https://git.openjdk.org/jdk/pull/23533#discussion_r1956664966


More information about the hotspot-compiler-dev mailing list