RFR: 8329032: C2 compiler register allocation support for APX EGPRs
Vladimir Kozlov
kvn at openjdk.org
Thu Jun 13 15:16:27 UTC 2024
On Wed, 1 May 2024 18:42:13 GMT, Jatin Bhateja <jbhateja at openjdk.org> wrote:
> Intel® Advanced Performance Extensions (Intel® APX) adds 16 new 64 bit general purpose register also known as Extended General Purpose Registers in IA-32e 64 bit mode.
>
> This initial patch adds following support:-
> 1) C2 compiler register allocation support.
> 2) State save restoration while transitioning from C2 JIT compiled code to runtime services.
> 3) Applicable extensions to native interface used by runtime for patching instruction.
>
> We plan to address C1 register support in subsequent patch as there are hard upper bound allocation limits
> (currently set to r11) imposed by existing implementation of linear scan algorithm after which it reserves
> remaining register for special purpose.
>
> Patch has been regressed over stand alone test points after merging with other APX support patches [1][2] under review.
>
> We plan to do thorough validation using [Intel's SDE](https://www.intel.com/content/www/us/en/download/684897/intel-software-development-emulator.html) during course of time and release incremental patches for bug fixes
> found during testing.
>
> [1] https://github.com/openjdk/jdk/pull/18476
> [2] https://github.com/openjdk/jdk/pull/18562
>
> PS: Intent of this draft PR is to facilitate validation of existing APX related PRs under review.
src/hotspot/cpu/x86/register_x86.hpp line 85:
> 83: if (UseAPX) {
> 84: return number_of_registers / 2;
> 85: }
Should check be reversed `!UseAPX`?
-------------
PR Review Comment: https://git.openjdk.org/jdk/pull/19042#discussion_r1631351591
More information about the hotspot-dev
mailing list