RFR: 8349579: jsvml.dll incorrect RDATA SEGMENT specification [v2]
Julian Waters
jwaters at openjdk.org
Tue Feb 11 08:07:11 UTC 2025
On Tue, 11 Feb 2025 03:44:27 GMT, Mohamed Issa <duke at openjdk.org> wrote:
>> A fix for incorrectly defined program segments in Windows SVML assembly.
>>
>> - Changes _READ_ to _READONLY_ in all math functions
>> - Now compliant with MASM x86 and x86_64 program segment [specification](https://learn.microsoft.com/en-us/cpp/assembler/masm/segment?view=msvc-170)
>>
>> The tier1 tests show the changes didn't introduce new failures.
>
> Mohamed Issa has updated the pull request incrementally with one additional commit since the last revision:
>
> Update full name
The microsoft toolchain still creates a literal section named _RDATA even with this change? Guess it's a fault in the toolchain and nothing to do with not conforming to the microsoft docs then. Still, like you said, it's good to follow what microsoft says in their docs
-------------
PR Comment: https://git.openjdk.org/jdk/pull/23503#issuecomment-2650067731
More information about the hotspot-compiler-dev
mailing list