RFR: 8329032: C2 compiler register allocation support for APX EGPRs [v3]

Vladimir Kozlov kvn at openjdk.org
Mon Jun 17 23:55:11 UTC 2024


On Mon, 17 Jun 2024 21:31:36 GMT, Jatin Bhateja <jbhateja at openjdk.org> wrote:

>> src/hotspot/cpu/x86/vm_version_x86.cpp line 422:
>> 
>>> 420:     __ movl(Address(rsi,12), rdx);
>>> 421: 
>>> 422: #if !defined(PRODUCT) && defined(_LP64)
>> 
>> Why it is still under `!PRODUCT`?  You need to check if OS supports it in product VM too. Or I missing something?
>
> Idea is to enable the APX support for product builds only after completion of planned features listed in JDK-8329030.
> 
> I have moved the check to [appropriate place](https://github.com/openjdk/jdk/pull/19042/files#diff-6ed856c57ddbe33e49883adb7c52ec51ed377e5f697dfd6d8bea505a97bfc5a5R3186)

Thank you for moving `PRODUCT` check and adding comment.

New save/restore methods should be under `#ifdef _LP64`. 32-bit build is broken again because of that.

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

PR Review Comment: https://git.openjdk.org/jdk/pull/19042#discussion_r1643560772


More information about the hotspot-compiler-dev mailing list