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