RFR: 8343452: Incorrect WINDOWS build variable is used in macroAssembler_x86.cpp

Christian Hagedorn chagedorn at openjdk.org
Tue Nov 5 15:57:28 UTC 2024


On Tue, 5 Nov 2024 12:21:58 GMT, theoweidmannoracle <duke at openjdk.org> wrote:

> From JBS:
> 
>> HotSpot VM build macro defines _WINDOWS env variable only.
>> call_clobbered_gp_registers() and call_clobbered_xmm_registers() incorrectly use WINDOWS (without underscore).
>> RSI and RDI are missing from list of clobbered register on windows due to that and may cause issue when calling native code which modifies them.
>> 
>> AMM register are less affected because all XMM registers are listed as clobbered.
>> 
>> The code was added by [JDK-8283327](https://bugs.openjdk.org/browse/JDK-8283327) changes in JDK 19.

Looks good!

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

Marked as reviewed by chagedorn (Reviewer).

PR Review: https://git.openjdk.org/jdk/pull/21901#pullrequestreview-2416046832


More information about the hotspot-dev mailing list