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